Skip to content
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

document workflow schema version #67

Closed
wants to merge 2 commits into from
Closed

document workflow schema version #67

wants to merge 2 commits into from

Conversation

mfleader
Copy link
Member

Changes introduced with this PR

  1. document workflow schema version
  2. document workflow schema and engine version compatibility
  3. fix broken multiple outputs link

By contributing to this repository, I agree to the contribution guidelines.


<h2>Version</h2>

The version section states which workflow schema version you are using to write your workflow. The latest supported workflow schema is `v0.1.0`.
Copy link
Member

Choose a reason for hiding this comment

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

It's going to be easy to overlook updating this static reference to the "latest supported workflow schema" in the future.

Copy link
Member Author

Choose a reason for hiding this comment

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

I had the same thought, but I don't think we need a dynamic solution until we have more than 2 workflow schema versions.

Copy link
Contributor

Choose a reason for hiding this comment

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

Are we aware of any good dynamic solution?

Copy link
Member Author

@mfleader mfleader Oct 24, 2023

Choose a reason for hiding this comment

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

I have done some research, and have not found a completely dynamic approach. From what I've seen folks still have to manually write down the mapping from file schema/format version to minimum engine version.

That mapping could be stored in a file which is read to dynamically generate the table.

Copy link
Contributor

Choose a reason for hiding this comment

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

I have done some research, and have not found a completely dynamic approach. From what I've seen folks still have to manually write down the mapping from file schema/format version to minimum engine version.

That mapping could be stored in a file which is read to dynamically generate the table.

True, in any kind of a "build" environment: but it's also nice to be able to just go to GitHub and read the markdown file directly, in which case it would just be a blank. (Unless GitHub markdown has someway to include a text file inline?)

It may not really be necessary to call out "the latest supported version" here. You already link to the matrix.md with a table; which at least reduces the manual update to one file instead of two.

Copy link
Member

Choose a reason for hiding this comment

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

I agree with @dbutenhof. I would recommend simply removing the explicit version reference here.


<h2>Version</h2>

The version section states which workflow schema version you are using to write your workflow. The latest supported workflow schema is `v0.1.0`.
Copy link
Contributor

Choose a reason for hiding this comment

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

I have done some research, and have not found a completely dynamic approach. From what I've seen folks still have to manually write down the mapping from file schema/format version to minimum engine version.

That mapping could be stored in a file which is read to dynamically generate the table.

True, in any kind of a "build" environment: but it's also nice to be able to just go to GitHub and read the markdown file directly, in which case it would just be a blank. (Unless GitHub markdown has someway to include a text file inline?)

It may not really be necessary to call out "the latest supported version" here. You already link to the matrix.md with a table; which at least reduces the manual update to one file instead of two.

@@ -0,0 +1,7 @@
# Workflow and Engine Compatibility Matrix

The workflow schema version uses the [semantic versioning 2.0.0 specification](https://semver.org/). In a workflow file definition, the schema version is prepended with a `v`. For example, `0.1.0` is written as `v0.1.0`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you want to explicitly recommend always creating a new workflow with the latest version? 😸

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, we need to be careful. Because the documentation's version needs to match what the documentation itself is recommending.

@mfleader
Copy link
Member Author

mfleader commented Feb 9, 2024

Changes merged in pr 70

@mfleader mfleader closed this Feb 9, 2024
@mfleader mfleader deleted the workflow-version branch February 9, 2024 17:40
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.

4 participants