Repository navigation
CI: stop Draft runner flood, scope main and firmware pushes, package on demand #158
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
Changes from all commits
e4966e4
f3c14d0
90d54a9
74ad858
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 |
|---|---|---|
|
|
@@ -4,7 +4,20 @@ on: | |
| push: | ||
| branches: | ||
| - main | ||
| paths: | ||
| - 'embedded/brcb_hard_profile/**' | ||
| - 'embedded/brcb_*.cpp' | ||
| - 'embedded/mms_brcb_*.cpp' | ||
| - 'include/ariec61850/mms/**' | ||
| - 'src/mms/**' | ||
| - 'include/ariec61850/control/**' | ||
| - 'src/control/**' | ||
| - 'include/ariec61850/asn1/**' | ||
| - 'src/asn1/**' | ||
| - 'tests/fixtures/scl/**' | ||
| - '.github/workflows/brcb-hard-profile-ci.yml' | ||
| pull_request: | ||
| types: [opened, synchronize, reopened, ready_for_review] | ||
|
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.
When a Ready PR is switched back to Draft while one of these expensive runs is active, GitHub emits the AGENTS.md reference: AGENTS.md:L15-L16 Useful? React with 👍 / 👎. |
||
| paths: | ||
| - 'embedded/brcb_hard_profile/**' | ||
| - 'embedded/brcb_*.cpp' | ||
|
|
@@ -27,6 +40,7 @@ concurrency: | |
|
|
||
| jobs: | ||
| brcb-hard-profile: | ||
| if: ${{ github.event_name != 'pull_request' || !github.event.pull_request.draft }} | ||
| name: BRCB Hard / ${{ matrix.compiler }} | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
|
|
||
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.
The new
mainpath filter omitssrc/osi/**,include/ariec61850/osi/**,src/acse/**, and their headers even thoughembedded/brcb_hard_profile/CMakeLists.txtdirectly compiles those sources into the lifecycle, connection, and negotiated-limits smoke targets. A merged OSI/ACSE change therefore no longer runs this specialized-fno-exceptionsBRCB gate, so failures unique to the hard profile can reachmaindespite the workflow previously running on every main push. Add every directly compiled dependency domain to this scope.AGENTS.md reference: AGENTS.md:L256-L261
Useful? React with 👍 / 👎.