-
Notifications
You must be signed in to change notification settings - Fork 163
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
Conversation
There was a problem hiding this 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:
- Remove the
.DS_Store
file, and add it to the root.gitignore
file - Remove the added tabs to the tables
Thanks!
There was a problem hiding this 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
Done - I also slightly changed the description of the "exit" flow directive |
There was a problem hiding this 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 ❤️
@geomagilles Could you please sign-off your last commits? |
serverlessworkflow#1082) * update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multiple YAML files Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(contributing): reverting pre-release versioning format Signed-off-by: Gilles Barbier <gilles@infinitic.io> --------- Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
…tion into fixDocs
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
…tion into fixDocs
it should be good - sorry I'm not used to signed commits and now they are messy - you should do a 'squash merge' |
Thanks @geomagilles! Unhappily, the DS_Store file has been included again 😞 |
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
…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 <gilles@infinitic.io> * fix(contributing): reverting pre-release versioning format Signed-off-by: Gilles Barbier <gilles@infinitic.io> --------- Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl): fix a few issues in dsl and dsl-reference Signed-off-by: Gilles Barbier <gilles@infinitic.io> * update .gitignore to include .DS_Store Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): correct table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): clarify flow directive descriptions Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): improve table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): clarify flow directive descriptions Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl): fix a few issues in dsl and dsl-reference Signed-off-by: Gilles Barbier <gilles@infinitic.io> * update .gitignore to include .DS_Store Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): correct table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): clarify flow directive descriptions Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): improve table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): clarify flow directive descriptions Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): improve table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * remove again .DS_Store Signed-off-by: Gilles Barbier <gilles@infinitic.io> --------- Signed-off-by: Gilles Barbier <gilles@infinitic.io> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>
…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 <gilles@infinitic.io> * fix(contributing): reverting pre-release versioning format Signed-off-by: Gilles Barbier <gilles@infinitic.io> --------- Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl): fix a few issues in dsl and dsl-reference Signed-off-by: Gilles Barbier <gilles@infinitic.io> * update .gitignore to include .DS_Store Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): correct table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): clarify flow directive descriptions Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): improve table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): clarify flow directive descriptions Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl): fix a few issues in dsl and dsl-reference Signed-off-by: Gilles Barbier <gilles@infinitic.io> * update .gitignore to include .DS_Store Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): correct table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): clarify flow directive descriptions Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): improve table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): clarify flow directive descriptions Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): improve table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * remove again .DS_Store Signed-off-by: Gilles Barbier <gilles@infinitic.io> --------- Signed-off-by: Gilles Barbier <gilles@infinitic.io> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>
* Fix #1076 - Make detail and title a runtime expression in the schema Signed-off-by: Ricardo Zanini <ricardozanini@gmail.com> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * Add string as anyOf Signed-off-by: Ricardo Zanini <ricardozanini@gmail.com> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * feat(schema): allow expressions in HTTP query and headers (#1080) Issue #1079 Signed-off-by: Alexandru Georoceanu <alex@navigheaza.ro> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * 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 <gilles@infinitic.io> * fix(contributing): reverting pre-release versioning format Signed-off-by: Gilles Barbier <gilles@infinitic.io> --------- Signed-off-by: Gilles Barbier <gilles@infinitic.io> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * 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 <gilles@infinitic.io> * fix(contributing): reverting pre-release versioning format Signed-off-by: Gilles Barbier <gilles@infinitic.io> --------- Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl): fix a few issues in dsl and dsl-reference Signed-off-by: Gilles Barbier <gilles@infinitic.io> * update .gitignore to include .DS_Store Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): correct table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): clarify flow directive descriptions Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): improve table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): clarify flow directive descriptions Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl): fix a few issues in dsl and dsl-reference Signed-off-by: Gilles Barbier <gilles@infinitic.io> * update .gitignore to include .DS_Store Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): correct table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): clarify flow directive descriptions Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): improve table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): clarify flow directive descriptions Signed-off-by: Gilles Barbier <gilles@infinitic.io> * fix(dsl-reference): improve table formatting Signed-off-by: Gilles Barbier <gilles@infinitic.io> * remove again .DS_Store Signed-off-by: Gilles Barbier <gilles@infinitic.io> --------- Signed-off-by: Gilles Barbier <gilles@infinitic.io> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * Fix broken link in README.md (#1084) Signed-off-by: Michal Broz <mbroz@us.ibm.com> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * Set task to support direct runtime expression Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * updated workflow schema along with table fix Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * updating the schema Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * addressing PR comment reviews Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * removing unwanted set example Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * Update dsl-reference.md Co-authored-by: Charles d'Avernas <charles.davernas@neuroglia.io> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * Update dsl-reference.md Co-authored-by: Charles d'Avernas <charles.davernas@neuroglia.io> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * Update dsl-reference.md Co-authored-by: Charles d'Avernas <charles.davernas@neuroglia.io> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * reverting back original set example change Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * Update dsl-reference.md Co-authored-by: Charles d'Avernas <charles.davernas@neuroglia.io> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * passing the set example as object Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * reverting to original Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * Add a new file that compares the Serverless Workflow to other popular DSLs Signed-off-by: Charles d'Avernas <charles.davernas@neuroglia.io> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> * Fix a the `catalog` link in the `use` section table (#1088) Signed-off-by: Charles d'Avernas <charles.davernas@neuroglia.io> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> --------- Signed-off-by: Ricardo Zanini <ricardozanini@gmail.com> Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com> Signed-off-by: Alexandru Georoceanu <alex@navigheaza.ro> Signed-off-by: Gilles Barbier <gilles@infinitic.io> Signed-off-by: Michal Broz <mbroz@us.ibm.com> Signed-off-by: Charles d'Avernas <charles.davernas@neuroglia.io> Co-authored-by: Ricardo Zanini <ricardozanini@gmail.com> Co-authored-by: Alexandru Georoceanu <alex@navigheaza.ro> Co-authored-by: Gilles Barbier <gilles@infinitic.io> Co-authored-by: mbroz2 <mbroz@us.ibm.com> Co-authored-by: Charles d'Avernas <charles.davernas@neuroglia.io>
Please specify parts of this PR update:
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: