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

Fix syntax for workflow update testing #206

Merged
merged 3 commits into from
Oct 29, 2024

Conversation

evankanderson
Copy link
Member

@evankanderson evankanderson commented Oct 29, 2024

Ref: https://github.com/mindersec/minder-rules-and-profiles/actions/runs/11562883909

Invalid workflow file: .github/workflows/update.yaml#L7
You have an error in your yaml syntax on line 7

Fixes #196

@evankanderson evankanderson requested a review from a team as a code owner October 29, 2024 20:38
@evankanderson evankanderson force-pushed the fix-actions branch 12 times, most recently from bae14d3 to f232262 Compare October 29, 2024 21:31
@evankanderson evankanderson force-pushed the fix-actions branch 9 times, most recently from a683fcc to dd5d7a9 Compare October 29, 2024 22:24
@evankanderson evankanderson force-pushed the fix-actions branch 2 times, most recently from a59bb6d to 526adaf Compare October 29, 2024 22:29
@evankanderson
Copy link
Member Author

In https://github.com/mindersec/minder-rules-and-profiles/actions/runs/11583451941/job/32248636750, I added a second file containing a ruletype yaml with a valid upgrade, and the action passed.

I think this is ready for review.

@evankanderson evankanderson merged commit e773ba1 into mindersec:main Oct 29, 2024
4 checks passed
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.

Add a CI job for validating ruletype upgrade compatibility
2 participants