-
Notifications
You must be signed in to change notification settings - Fork 643
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
Update syntax docs #5615
Update syntax docs #5615
Conversation
Signed-off-by: Ben Sherman <bentshermann@gmail.com>
❌ Deploy Preview for nextflow-docs-staging failed. Why did it fail? →
|
Not sure why the netlify build is failing. But would be good to merge these changes this week to go along with the vscode extension released today. |
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.
Minor comments and suggestions.
Co-authored-by: Chris Hakkaart <chris.hakkaart@seqera.io> Signed-off-by: Ben Sherman <bentshermann@gmail.com>
Signed-off-by: Ben Sherman <bentshermann@gmail.com>
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.
The tests are failing and I can't see any reason why. The text itself is good to go. I don't have access to the netlify logs to view what's going wrong.
Not sure why netlify isn't working, but it is not required. If you can just approve then I can merge |
Add some notes to the syntax migration guide, as well as some updates for v1.0.3 of the language server.