Skip to content

Fix the description of flow directives and addressed few issues in dsl and dsl-reference #1083

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 18 commits into from
Feb 24, 2025

Conversation

geomagilles
Copy link
Contributor

Please specify parts of this PR update:

  • Specification
  • Schema
  • Examples
  • Extensions
  • Use Cases
  • Community
  • CTK
  • Other

Discussion or Issue link:
https://cloud-native.slack.com/archives/CMBAZQSEP/p1739812901298019
https://cloud-native.slack.com/archives/CMBAZQSEP/p1739085346279299

What this PR does:
Fix a few issues with the docs

Additional information:

Copy link
Member

@ricardozanini ricardozanini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @geomagilles!

Can you please consider:

  1. Remove the .DS_Store file, and add it to the root .gitignore file
  2. Remove the added tabs to the tables

Thanks!

Copy link
Member

@cdavernas cdavernas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, aside from a couple of changes, space and tabs related

@geomagilles
Copy link
Contributor Author

Done - I also slightly changed the description of the "exit" flow directive

Copy link
Member

@cdavernas cdavernas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Thanks @geomagilles ❤️

@cdavernas
Copy link
Member

@geomagilles Could you please sign-off your last commits?

@cdavernas cdavernas changed the title fix(dsl): fix a few issues in dsl and dsl-reference Fix a few issues in dsl and dsl-reference Feb 21, 2025
@cdavernas cdavernas changed the title Fix a few issues in dsl and dsl-reference Fix tge description of flow directives and made a few issues in dsl and dsl-reference Feb 21, 2025
@cdavernas cdavernas changed the title Fix tge description of flow directives and made a few issues in dsl and dsl-reference Fix the description of flow directives and made a few issues in dsl and dsl-reference Feb 21, 2025
@cdavernas cdavernas changed the title Fix the description of flow directives and made a few issues in dsl and dsl-reference Fix the description of flow directives and addressed few issues in dsl and dsl-reference Feb 21, 2025
serverlessworkflow#1082)

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multiple YAML files

Signed-off-by: Gilles Barbier <[email protected]>

* fix(contributing): reverting pre-release versioning format

Signed-off-by: Gilles Barbier <[email protected]>

---------

Signed-off-by: Gilles Barbier <[email protected]>
@geomagilles
Copy link
Contributor Author

geomagilles commented Feb 22, 2025

it should be good - sorry I'm not used to signed commits and now they are messy - you should do a 'squash merge'

@cdavernas
Copy link
Member

Thanks @geomagilles! Unhappily, the DS_Store file has been included again 😞

Signed-off-by: Gilles Barbier <[email protected]>
@ricardozanini ricardozanini merged commit a4fa9ca into serverlessworkflow:main Feb 24, 2025
3 checks passed
nyamathshaik pushed a commit to nyamathshaik/specification that referenced this pull request Mar 10, 2025
…l and dsl-reference (serverlessworkflow#1083)

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multip… (serverlessworkflow#1082)

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multiple YAML files

Signed-off-by: Gilles Barbier <[email protected]>

* fix(contributing): reverting pre-release versioning format

Signed-off-by: Gilles Barbier <[email protected]>

---------

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl): fix a few issues in dsl and dsl-reference

Signed-off-by: Gilles Barbier <[email protected]>

* update .gitignore to include .DS_Store

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): correct table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): improve table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl): fix a few issues in dsl and dsl-reference

Signed-off-by: Gilles Barbier <[email protected]>

* update .gitignore to include .DS_Store

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): correct table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): improve table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): improve table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* remove again .DS_Store

Signed-off-by: Gilles Barbier <[email protected]>

---------

Signed-off-by: Gilles Barbier <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>
nyamathshaik pushed a commit to nyamathshaik/specification that referenced this pull request Mar 19, 2025
…l and dsl-reference (serverlessworkflow#1083)

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multip… (serverlessworkflow#1082)

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multiple YAML files

Signed-off-by: Gilles Barbier <[email protected]>

* fix(contributing): reverting pre-release versioning format

Signed-off-by: Gilles Barbier <[email protected]>

---------

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl): fix a few issues in dsl and dsl-reference

Signed-off-by: Gilles Barbier <[email protected]>

* update .gitignore to include .DS_Store

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): correct table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): improve table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl): fix a few issues in dsl and dsl-reference

Signed-off-by: Gilles Barbier <[email protected]>

* update .gitignore to include .DS_Store

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): correct table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): improve table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): improve table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* remove again .DS_Store

Signed-off-by: Gilles Barbier <[email protected]>

---------

Signed-off-by: Gilles Barbier <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>
ricardozanini added a commit that referenced this pull request Mar 19, 2025
* Fix #1076 - Make detail and title a runtime expression in the schema

Signed-off-by: Ricardo Zanini <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>

* Add string as anyOf

Signed-off-by: Ricardo Zanini <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>

* feat(schema): allow expressions in HTTP query and headers (#1080)

Issue #1079

Signed-off-by: Alexandru Georoceanu <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multip… (#1082)

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multiple YAML files

Signed-off-by: Gilles Barbier <[email protected]>

* fix(contributing): reverting pre-release versioning format

Signed-off-by: Gilles Barbier <[email protected]>

---------

Signed-off-by: Gilles Barbier <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>

* Fix the description of flow directives and addressed few issues in dsl and dsl-reference (#1083)

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multip… (#1082)

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multiple YAML files

Signed-off-by: Gilles Barbier <[email protected]>

* fix(contributing): reverting pre-release versioning format

Signed-off-by: Gilles Barbier <[email protected]>

---------

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl): fix a few issues in dsl and dsl-reference

Signed-off-by: Gilles Barbier <[email protected]>

* update .gitignore to include .DS_Store

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): correct table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): improve table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl): fix a few issues in dsl and dsl-reference

Signed-off-by: Gilles Barbier <[email protected]>

* update .gitignore to include .DS_Store

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): correct table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): improve table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <[email protected]>

* fix(dsl-reference): improve table formatting

Signed-off-by: Gilles Barbier <[email protected]>

* remove again .DS_Store

Signed-off-by: Gilles Barbier <[email protected]>

---------

Signed-off-by: Gilles Barbier <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>

* Fix broken link in README.md (#1084)

Signed-off-by: Michal Broz <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>

* Set task to support direct runtime expression

Signed-off-by: Nyamath Shaik <[email protected]>

* updated workflow schema along with table fix

Signed-off-by: Nyamath Shaik <[email protected]>

* updating the schema

Signed-off-by: Nyamath Shaik <[email protected]>

* addressing PR comment reviews

Signed-off-by: Nyamath Shaik <[email protected]>

* removing unwanted set example

Signed-off-by: Nyamath Shaik <[email protected]>

* Update dsl-reference.md

Co-authored-by: Charles d'Avernas <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>

* Update dsl-reference.md

Co-authored-by: Charles d'Avernas <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>

* Update dsl-reference.md

Co-authored-by: Charles d'Avernas <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>

* reverting back original set example change

Signed-off-by: Nyamath Shaik <[email protected]>

* Update dsl-reference.md

Co-authored-by: Charles d'Avernas <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>

* passing the set example as object

Signed-off-by: Nyamath Shaik <[email protected]>

* reverting to original

Signed-off-by: Nyamath Shaik <[email protected]>

* Add a new file that compares the Serverless Workflow to other popular DSLs

Signed-off-by: Charles d'Avernas <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>

* Fix a the `catalog` link in the `use` section table (#1088)

Signed-off-by: Charles d'Avernas <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>

---------

Signed-off-by: Ricardo Zanini <[email protected]>
Signed-off-by: Nyamath Shaik <[email protected]>
Signed-off-by: Alexandru Georoceanu <[email protected]>
Signed-off-by: Gilles Barbier <[email protected]>
Signed-off-by: Michal Broz <[email protected]>
Signed-off-by: Charles d'Avernas <[email protected]>
Co-authored-by: Ricardo Zanini <[email protected]>
Co-authored-by: Alexandru Georoceanu <[email protected]>
Co-authored-by: Gilles Barbier <[email protected]>
Co-authored-by: mbroz2 <[email protected]>
Co-authored-by: Charles d'Avernas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants