-
Notifications
You must be signed in to change notification settings - Fork 21
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
generate inputs-outputs doc #210
Conversation
- remove static docs/inputs-outputs.md - gitignore docs/inputs-outputs.md - add script to generate docs/inputs-outputs.md dynamically on build
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.
It would be nice if we could validate the action.yml before publishing a release.
Hope so :)
From filed |
I agree, and we've been doing that for the most part. Some minimum-versions that are set to a patch bump are the result of bug fixes or patched behavior. For example, we didn't do a minor version bump for new input no-lgtm because that was a flag to restore previous thread-comments behavior while retaining the capability introduced in the minor version bump for step-summary. |
This reverts commit b5db5b1.
Does the same thing as #210 without maligning action.yml
Does the same thing as #210 without maligning action.yml
Caution
This adds metadata/fields to the action.yml that is not used by github actions.
I don't see anything in the github docs that forbids the use of superfluous fields in action.yml.
It would be nice if we could validate the action.yml before publishing a release.