Skip to content

chore(security): add Dependabot config#2

Merged
jml6m merged 2 commits into
masterfrom
chore/security-audit-setup
Jun 7, 2026
Merged

chore(security): add Dependabot config#2
jml6m merged 2 commits into
masterfrom
chore/security-audit-setup

Conversation

@jml6m

@jml6m jml6m commented Jun 7, 2026

Copy link
Copy Markdown
Owner

Part of the org-wide repo security audit.

This repo has no package.json (front-end assets are loaded via CDN), so the npm ecosystem block is intentionally commented out as a template for future use. Only github-actions is active today, which will keep any future workflow Action versions patched automatically.

Part of the org-wide repo security audit.

This repo has no package.json (front-end assets are loaded via CDN),
so the npm ecosystem block is intentionally commented out as a
template for future use. Only github-actions is active today, which
will keep any future workflow Action versions patched automatically.

Copilot AI 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.

Pull request overview

Adds a Dependabot configuration as part of the org-wide security audit, enabling automated dependency update PRs for GitHub Actions while leaving a commented npm template for a potential future migration to a bundled/npm-managed frontend.

Changes:

  • Add .github/dependabot.yml with weekly checks for the github-actions ecosystem.
  • Configure Dependabot PR metadata (commit message prefix/scope) and include a commented-out npm configuration template for future use.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/dependabot.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Joseph Lewkovich <brzt3x@gmail.com>
@jml6m

jml6m commented Jun 7, 2026

Copy link
Copy Markdown
Owner Author

@copilot why does it say:

Merging is blocked
Commits must have verified signatures.

Copilot AI commented Jun 7, 2026

Copy link
Copy Markdown

@copilot why does it say:

...

Because the PR is targeting the protected master branch, and that branch appears to require verified commit signatures. That check is about GitHub-recognized GPG/SSH/S/MIME signatures on the commits in the PR—not the Signed-off-by: line in a commit message. So if even one commit in this PR isn’t cryptographically verified by GitHub, merge gets blocked.

Copilot finished work on behalf of jml6m June 7, 2026 14:17
@jml6m

jml6m commented Jun 7, 2026

Copy link
Copy Markdown
Owner Author

@copilot Yes but If I'm having you do PRs and changing the code on those PRs...why aren't you following the repo rules? Or is some setting misconfigured?

Copilot AI commented Jun 7, 2026

Copy link
Copy Markdown

@copilot Yes but If I'm having you do PRs and changing the code on those PRs...why aren't you following the repo rules? Or is some setting misconfigured?

It looks more like a limitation of the commit path than a bad repo rule. The branch rule is working as configured, but the original PR commit (55aa078) appears to have been created without a GitHub-verifiable GPG/SSH/S/MIME signature, so it doesn’t satisfy “verified signatures required.” The later web-flow suggestion commit does show a cryptographic signature. So the mismatch is: I was able to create/update the PR, but that path didn’t guarantee a verified-signed commit. To satisfy the rule, the unsigned commit would need to be recreated with a verified signing key (or the rule/exemptions adjusted for this workflow).

Copilot finished work on behalf of jml6m June 7, 2026 14:20
@jml6m jml6m enabled auto-merge (squash) June 7, 2026 14:20
@jml6m jml6m disabled auto-merge June 7, 2026 14:20
@jml6m jml6m merged commit 802dae3 into master Jun 7, 2026
5 checks passed
@jml6m jml6m deleted the chore/security-audit-setup branch June 7, 2026 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants