Skip to content

Add CI workflow to validate action.yml #2

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 2 commits into from
May 4, 2021
Merged

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented May 3, 2021

On every push or pull request that affects the action.yml GitHub Actions metadata file, and periodically, validate action.yml against its JSON schema.

@per1234 per1234 added type: enhancement Proposed improvement topic: infrastructure Related to project infrastructure labels May 3, 2021
@per1234 per1234 force-pushed the validate-action-yml branch from 47b5365 to 7531b99 Compare May 4, 2021 04:05
@per1234 per1234 requested a review from silvanocerza May 4, 2021 08:11
Copy link
Contributor

@rsora rsora left a comment

Choose a reason for hiding this comment

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

Outstanding

per1234 added 2 commits May 4, 2021 03:26
On every push or pull request that affects the action.yml GitHub Actions metadata file, and periodically, validate
action.yml against its JSON schema.
Even though it obviously works fine without it, this is a required key according to the official GitHub Actions
documentation:

https://docs.github.com/en/actions/creating-actions/metadata-syntax-for-github-actions#inputsinput_idrequired

Accordingly, it is required by the JSON schema.
@per1234 per1234 force-pushed the validate-action-yml branch from 7531b99 to 6829272 Compare May 4, 2021 10:26
@per1234 per1234 merged commit c65074e into main May 4, 2021
@per1234 per1234 deleted the validate-action-yml branch May 4, 2021 10:30
@per1234 per1234 self-assigned this Nov 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants