Skip to content

Sync Dev with main - #15

Merged
reactive-firewall merged 9 commits into
devfrom
main
Jul 28, 2026
Merged

Sync Dev with main#15
reactive-firewall merged 9 commits into
devfrom
main

Conversation

@reactive-firewall

@reactive-firewall reactive-firewall commented Jul 28, 2026

Copy link
Copy Markdown
Owner

Patch Notes

Just bringing dev up-to-date with recent version bumps.

Impacted GHIs:

  • None

Included & Superseded PR/MRs


Summary by CodeRabbit

  • Chores
    • Updated artifact upload and SARIF publishing workflows to newer action versions.
    • Maintains compatibility with current GitHub workflow tooling.

reactive-firewall and others added 9 commits September 8, 2024 23:08
…ity features

> [!NOTE]
> 
> Due to the backup, upstream with
[actions/starter-workflows#2497](actions/starter-workflows#2497)
not yet resolved, this PR will include at-least two minor version bumps:
> 
> *
[v2.2](637c5c4)
@
[637c5c4](637c5c4)
> *
[v2.3](f8cf05e)
@
[f8cf05e](f8cf05e)

---
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3 to 4.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@v3...v4)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: '4'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 5.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v4...v5)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
* dependabot/github_actions/main/actions/upload-artifact-5:
  [UPDATE] (deps): Bump actions/upload-artifact from 4 to 5
  [UPDATE] (deps): Bump github/codeql-action from 3 to 4
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 5 to 6.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v5...v6)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Merge branch 'dependabot/github_actions/main/actions/upload-artifact-6'

* dependabot/github_actions/main/actions/upload-artifact-6:
  [UPDATE] (deps): Bump actions/upload-artifact from 5 to 6
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 6 to 7.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v6...v7)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@reactive-firewall reactive-firewall self-assigned this Jul 28, 2026
@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The composite workflow updates its artifact upload action from v4 to v7 and its SARIF upload action from v3 to v4. Existing artifact naming, paths, and overwrite settings remain unchanged.

Changes

Scan output publishing

Layer / File(s) Summary
Upload action version upgrades
action.yml
The artifact upload step now uses actions/upload-artifact@v7, and the SARIF publishing step uses github/codeql-action/upload-sarif@v4.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title is relevant because this PR syncs dev with main, though it is broader than the specific action version bumps.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch main

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@reactive-firewall reactive-firewall added the duplicate This issue or pull request already exists label Jul 28, 2026
@reactive-firewall reactive-firewall linked an issue Jul 28, 2026 that may be closed by this pull request

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
action.yml (1)

127-127: 🩺 Stability & Availability | 🔵 Trivial

Document runner runtime requirements for the upgraded upload steps.

Both upgraded action references are Node.js 24 runtimes; self-hosted runners before GitHub Actions Runner 2.327.1 may fail before these uploads execute. Make the minimum runner version explicit in the action’s README or usage documentation.

Also applies to line 134.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@action.yml` at line 127, Document the minimum GitHub Actions Runner version
required by the Node.js 24 upload-artifact steps referenced in action.yml,
including both upgraded action references. Update the action’s README or usage
documentation to state that self-hosted runners must be version 2.327.1 or
newer.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@action.yml`:
- Line 127: Document the minimum GitHub Actions Runner version required by the
Node.js 24 upload-artifact steps referenced in action.yml, including both
upgraded action references. Update the action’s README or usage documentation to
state that self-hosted runners must be version 2.327.1 or newer.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 4eefb406-2ab9-41d1-ae45-b99695cb8856

📥 Commits

Reviewing files that changed from the base of the PR and between c56ff8d and 88ecea1.

📒 Files selected for processing (1)
  • action.yml

@reactive-firewall
reactive-firewall merged commit 919c5b3 into dev Jul 28, 2026
22 checks passed
reactive-firewall added a commit that referenced this pull request Jul 28, 2026
# Patch Notes

Back-ports 2 additional trivial updates

## Impacted GHIs:

 * Contributes to work on #10 (additional updates)
 
## Included & Superseded PR/MRs

 * Includes #5
 * Includes & Supersedes #6
 * Includes & Supersedes #7
 * Includes & Supersedes #11
 * Includes & Supersedes #12
 * Includes & Supersedes #13
 * Includes & Supersedes #14
 * Includes #15

---
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

duplicate This issue or pull request already exists

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Out-of sync with upstream

1 participant