Skip to content

ci: consolidate and improve GitHub Actions workflows#477

Merged
mramotowski merged 1 commit into
open-edge-platform:masterfrom
mramotowski:mramotowski/gr305
Jan 30, 2026
Merged

ci: consolidate and improve GitHub Actions workflows#477
mramotowski merged 1 commit into
open-edge-platform:masterfrom
mramotowski:mramotowski/gr305

Conversation

@mramotowski

@mramotowski mramotowski commented Jan 29, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Consolidate and improve GitHub Actions workflows:

  • Merge test_accuracy.yml and test_precommit.yml into pre_commit.yml
  • Add concurrency control to prevent duplicate workflow runs
  • Unify security scans to handle both PRs and scheduled runs
  • Move permissions to workflow level with job-level overrides
  • Standardize YAML formatting and schedule cron times
  • Use YAML anchors to reduce duplication in pre_commit.yml
  • Update deprecated set-output syntax to use GITHUB_OUTPUT
  • Simplify CodeQL matrix by moving build-mode to step level
  • Pin uv version to 0.9.27 for reproducibility
  • Add blank lines between steps for improved readability
  • Replace regex-match action with native bash

Fixes
https://github.com/intel-innersource/applications.ai.geti.geti-release/issues/305
#387

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

@mramotowski mramotowski requested a review from a team as a code owner January 29, 2026 09:38
@github-actions github-actions Bot added the build Related to build scripts label Jan 29, 2026
@mramotowski mramotowski force-pushed the mramotowski/gr305 branch 8 times, most recently from 5ac467d to 74a9f02 Compare January 29, 2026 13:12

@AlexanderBarabanov AlexanderBarabanov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for the effort. I have a few minor suggestions.

Comment thread .github/workflows/security-scan.yml
Comment thread .github/workflows/publish.yaml Outdated
Comment thread .github/workflows/security-scan.yml
Comment thread .github/workflows/security-scan.yml Outdated
Comment thread .github/workflows/security-scan.yml Outdated
- Merge test_accuracy.yml and test_precommit.yml into pre_commit.yml
- Add concurrency control to prevent duplicate workflow runs
- Unify security scans to handle both PRs and scheduled runs
- Move permissions to workflow level with job-level overrides
- Standardize YAML formatting and schedule cron times
- Use YAML anchors to reduce duplication in pre_commit.yml
- Update deprecated set-output syntax to use GITHUB_OUTPUT
- Simplify CodeQL matrix by moving build-mode to step level
- Pin uv version to 0.9.27 for reproducibility
- Add blank lines between steps for improved readability
- Replace regex-match action with native bash

Signed-off-by: mramotowski <maciej.ramotowski@intel.com>
@mramotowski mramotowski enabled auto-merge January 30, 2026 13:32
@mramotowski mramotowski added this pull request to the merge queue Jan 30, 2026
Merged via the queue into open-edge-platform:master with commit 7ff62cf Jan 30, 2026
91 of 99 checks passed
@mramotowski mramotowski deleted the mramotowski/gr305 branch January 30, 2026 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Related to build scripts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants