Skip to content

fix: avoid pinning first-party GitHub action refs#55

Merged
lumirlumir merged 1 commit intomainfrom
fix/skip-first-party-action-pinning
Mar 27, 2026
Merged

fix: avoid pinning first-party GitHub action refs#55
lumirlumir merged 1 commit intomainfrom
fix/skip-first-party-action-pinning

Conversation

@Pixel998
Copy link
Copy Markdown
Contributor

Prerequisites checklist

AI acknowledgment

  • I did not use AI to generate this PR.
  • (If the above is not checked) I have reviewed the AI-generated content before submitting.

What is the purpose of this pull request?

This PR prevents Renovate from opening PRs like eslint/eslint.org#984, where a first-party reusable workflow ref is pinned to a commit SHA.

What changes did you make? (Give an overview)

Updated the Renovate base config so the rule that pins third-party GitHub Action digests no longer applies to eslint/** dependencies.

Related Issues

Is there anything you'd like reviewers to focus on?

@eslint-github-bot eslint-github-bot bot added the bug Something isn't working label Mar 26, 2026
@eslintbot eslintbot added this to Triage Mar 26, 2026
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Mar 26, 2026
@nzakas nzakas moved this from Needs Triage to Second Review Needed in Triage Mar 26, 2026
Copy link
Copy Markdown
Member

@nzakas nzakas left a comment

Choose a reason for hiding this comment

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

I think this is a good idea. @lumirlumir @fasttime do you agree?

@fasttime
Copy link
Copy Markdown
Member

Makes sense 👍

@lumirlumir lumirlumir added the accepted There is consensus among the team that this change meets the criteria for inclusion label Mar 27, 2026
Copy link
Copy Markdown
Member

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

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

Labels

accepted There is consensus among the team that this change meets the criteria for inclusion bug Something isn't working

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

5 participants