-
Notifications
You must be signed in to change notification settings - Fork 14
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
chore(ci): do not run ci on pushes to main or dev #2027
Changes from all commits
d1715d8
a8e198f
f5afad9
e255a35
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,11 +12,11 @@ | |
name: "CodeQL" | ||
|
||
on: | ||
push: | ||
branches: [main] | ||
# push: | ||
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Disabling CodeQL analysis on push to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Commenting out lining 11 and 12 results in all branches triggering a workflow run when a pull request is made. Depending on your project's branch strategy this may be excessive and cause unnecessary use of action minutes. |
||
# branches: [main] | ||
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
|
||
pull_request: | ||
# The branches below must be a subset of the branches above | ||
branches: [main] | ||
# branches: [main] | ||
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
|
||
schedule: | ||
- cron: "36 16 * * 4" | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,8 +2,12 @@ name: Node.js | |
|
||
on: | ||
push: | ||
branches: | ||
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Skipping push event actions on 'main' branch could lead to untested code getting into 'main', which is generally considered as production-ready or release branch. We should ensure all tests and actions are still performed on push to these 'main' branch to prevent any issues with the production code. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. On line 28, the 'branches' trigger is set to exclude 'main' and 'dev'. The reason should be clarified and relevant documentation updated accordingly. |
||
- '!main' | ||
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
|
||
- '!dev' | ||
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
|
||
pull_request: | ||
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
|
||
workflow_dispatch: | ||
drazisil marked this conversation as resolved.
Show resolved
Hide resolved
|
||
merge_group: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The 'merge_group:' key does not exist in GitHub action workflows. Carefully review the workflow yaml configurations for valid syntax. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. On line 33, 'merge_group' is not a known event type in GitHub Actions. Please correct this to a valid trigger event type. |
||
|
||
env: | ||
MCO_LOG_LEVEL: warn | ||
|
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.
Commenting out 'push' events on line 9 and 10 suggests that the workflow will not run on push events. Ensuring regular code scans are run can be important for maintaining code quality; you may want to reconsider this change.