Skip to content

Fix ESLint plugin publishing provenance - #8625

Merged
gonzaloriestra merged 1 commit into
mainfrom
gonzalo/eslint-plugin-provenance
Sep 23, 2026
Merged

gonzaloriestra merged 1 commit into
mainfrom
gonzalo/eslint-plugin-provenance

Conversation

@gonzaloriestra

@gonzaloriestra gonzaloriestra commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

After #8576 bumped @shopify/eslint-plugin-cli to the unpublished version 4.0.0, the release workflow started attempting to publish it. npm rejects the package with HTTP 422 because its missing repository.url cannot be matched to the GitHub Actions provenance. See the failed release.

WHAT is this pull request doing?

Add the repository URL and package directory, following the other published packages, so npm can validate the package's provenance.

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • The change is user-facing — I've identified the correct bump type (patch for bug fixes · minor for new features · major for breaking changes) and added a changeset with pnpm changeset add

@gonzaloriestra
gonzaloriestra requested a review from a team as a code owner September 23, 2026 08:36
@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Sep 23, 2026
@gonzaloriestra
gonzaloriestra added this pull request to the merge queue Sep 23, 2026
Merged via the queue into main with commit b65ce73 Sep 23, 2026
71 of 80 checks passed
@gonzaloriestra
gonzaloriestra deleted the gonzalo/eslint-plugin-provenance branch September 23, 2026 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants