Skip to content

CI Security and Style fixes#2034

Open
BenBE wants to merge 5 commits into
htop-dev:mainfrom
BenBE:ci-sec-fix
Open

CI Security and Style fixes#2034
BenBE wants to merge 5 commits into
htop-dev:mainfrom
BenBE:ci-sec-fix

Conversation

@BenBE

@BenBE BenBE commented Jun 24, 2026

Copy link
Copy Markdown
Member

Fixes #2032

@coderabbitai : Please suggest further issues in the CI setup now. This PR will contain cleanup of the CI stuff, so let's concentrate CI-related changes instead of sprinkling them all over the place.

@BenBE BenBE added this to the 3.6.0 milestone Jun 24, 2026
@BenBE BenBE added code quality ♻️ Code quality enhancement build system 🔧 Affects the build system rather then the user experience security 👮 Issues with security implications dependencies Pull requests that update a dependency file labels Jun 24, 2026
@BenBE BenBE changed the title Ci Security and Style fixes CI Security and Style fixes Jun 24, 2026
@coderabbitai

coderabbitai Bot commented Jun 24, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 79772062-2098-4486-815a-142bfb058420

📥 Commits

Reviewing files that changed from the base of the PR and between c67b1e8 and bf9d40f.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

The CI workflow updates checkout steps to stop persisting credentials across several jobs, replaces deprecated LLVM apt-key repository setup with scoped keyrings in four Ubuntu Clang jobs, changes one unsupported-platform shell substitution to $(uname -m), rewrites BSD and Solaris VM build jobs using vmactions/*-vm, and refreshes lint job wiring.

Assessment against linked issues

Objective Addressed Explanation
Replace apt-key/add-apt-repository with signed-by keyrings in all four Clang jobs [#2032]
Replace backtick command substitutions with $() in the workflow [#2032] The unsupported-platform --host substitution is updated, but the four Clang-job ubuntu_codename=\...`` substitutions are not shown in the provided change summary.

Assessment against linked issues: Out-of-scope changes

Code Change Explanation
Rewritten BSD and Solaris VM build jobs (.github/workflows/ci.yml, lines 288-416) The linked issue only requests LLVM apt-key removal and shell-style fixes. These VM job changes are unrelated to those objectives.
Checkout credential persistence changes and lint job wiring updates (.github/workflows/ci.yml, lines 18-19, 74-75, 125-126, 145-146, 250-251, 271-272, 417-433) These workflow adjustments are not part of the linked issue requirements.

Poem

Old keys depart; new keyrings sing,
Shell syntax leans on calmer wings.
BSDs and Solaris march anew,
While lint and checkout tidy through.
The workflow hums, more crisp than before.


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.

@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.

Actionable comments posted: 3


ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: aaa53919-1d3b-4426-859c-9bb82d08de88

📥 Commits

Reviewing files that changed from the base of the PR and between f137d5a and c67b1e8.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build system 🔧 Affects the build system rather then the user experience code quality ♻️ Code quality enhancement dependencies Pull requests that update a dependency file security 👮 Issues with security implications

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: Replace deprecated apt-key and fix shell style in workflow jobs

1 participant