Skip to content
This repository was archived by the owner on Feb 3, 2026. It is now read-only.

Create workflow to autofix dependabot PR#9

Merged
Cellule merged 6 commits intomasterfrom
dependabot-autofix
Jun 27, 2025
Merged

Create workflow to autofix dependabot PR#9
Cellule merged 6 commits intomasterfrom
dependabot-autofix

Conversation

@Cellule
Copy link
Copy Markdown
Member

@Cellule Cellule commented Jun 23, 2025

This change is Reviewable

@Cellule Cellule self-assigned this Jun 23, 2025
@Cellule Cellule force-pushed the dependabot-autofix branch from 8ea9707 to 8530f85 Compare June 23, 2025 19:33
@Cellule Cellule marked this pull request as ready for review June 23, 2025 19:37
@github-actions github-actions bot enabled auto-merge (squash) June 23, 2025 19:38
@Cellule Cellule disabled auto-merge June 23, 2025 19:38
@Cellule Cellule requested a review from Copilot June 23, 2025 20:01
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a new environment variable to bypass Yarn’s --immutable lockfile enforcement, updates the plugin code and documentation accordingly, and adds a GitHub Actions workflow to auto-fix Dependabot PRs.

  • Add WORKSPACE_LOCKFILE_FORCE_WRITE check in both source and bundled code to control lockfile writes under --immutable
  • Update README to document the new environment variable
  • Add .github/workflows/dependabot-auto-fix.yml to automatically install, fix lockfiles, commit, and auto-merge Dependabot PRs

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
src/index.ts Introduce isImmutable flag based on WORKSPACE_LOCKFILE_FORCE_WRITE
bundles/@yarnpkg/plugin-workspace-lockfile.js Mirror the new env var logic in the bundled plugin
README.md Document WORKSPACE_LOCKFILE_FORCE_WRITE usage
.github/workflows/dependabot-auto-fix.yml New workflow to auto-fix and auto-merge Dependabot pull requests
Comments suppressed due to low confidence (2)

README.md:124

  • It may be helpful to explicitly state the default behavior (when the variable is absent or set to anything other than true) so users know the env var must be exactly true.
### `WORKSPACE_LOCKFILE_FORCE_WRITE`

.github/workflows/dependabot-auto-fix.yml:16

  • Add with: fetch-depth: 0 under the checkout step to ensure full history is available for the auto-merge and squash operation.
      - uses: actions/checkout@v4

Copy link
Copy Markdown
Contributor

@ThomasTrepanier ThomasTrepanier left a comment

Choose a reason for hiding this comment

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

Reviewed 4 of 4 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @AdrienPoupa, @alexisloiselle, @Cellule, @Cloudsky01, @jpineault, @ptessier, and @Urik)

@Cellule Cellule merged commit 215909d into master Jun 27, 2025
2 of 3 checks passed
@Cellule Cellule deleted the dependabot-autofix branch June 27, 2025 16:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants