-
Notifications
You must be signed in to change notification settings - Fork 3
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
Do not trigger CI when .md files are changed #290
base: main
Are you sure you want to change the base?
Conversation
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #290 +/- ##
=======================================
Coverage 77.97% 77.97%
=======================================
Files 21 21
Lines 990 990
=======================================
Hits 772 772
Misses 218 218 ☔ View full report in Codecov by Sentry. |
… pass so PR can be accepted
When merged, I had to test it further |
…ild and test but check-all-green can still pass so PR could be merged
if [ "$all_match" = true ]; then | ||
echo "skip=true" >> "$GITHUB_OUTPUT" | ||
else | ||
echo "skip=false" >> "$GITHUB_OUTPUT" | ||
fi |
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.
Can this be replaced with echo "skip="$all_match >> "$GITHUB_OUTPUT"
?
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.
I did it in this way because we might need to emit some data for collection in case build is skipped. I'll check with @vmilosevic and change it accordingly.
files to ignore: *.md, *.gitignore, LICENSE
closes #295