Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): manually update dependencies 30/11/2021 #436

Merged
merged 3 commits into from
Nov 30, 2021

Conversation

simonbyford
Copy link
Contributor

@simonbyford simonbyford commented Nov 30, 2021

What does this change?

Why?

Dependabot doesn't support pnpm which means most of our dependabot PRs have been broken since we introduced pnpm on 18th October. They are failing with the error:

 ERR_PNPM_OUTDATED_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up-to-date with package.json

This is a known issue: dependabot/dependabot-core#1736

To temporarily resolve this, I closed a bunch of affected PRs (see below) and ran pnpm update --latest manually. This bumped all the relevant dependencies in one go.

Open questions:

  • is there a workaround to get pnpm working with dependabot?
  • if not, is it worth ditching either dependabot or pnpm on this repo?
  • what would the main drawbacks of simply running pnpm update --latest every week instead of relying on the usual dependabot workflow? Off the top of my head: having individual PRs for each dependency gives us more control and makes issues easier to isolate

@simonbyford simonbyford force-pushed the manual-deps-update-30-11-2021 branch from 8feee8a to 803bce2 Compare November 30, 2021 11:03
Bumps [actions/cache](https://github.com/actions/cache) from 2.1.6 to 2.1.7.
- [Release notes](https://github.com/actions/cache/releases)
- [Commits](actions/cache@v2.1.6...v2.1.7)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Copy link
Contributor

@mxdvl mxdvl left a comment

Choose a reason for hiding this comment

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

If this works, it’s great.

If we cannot get @dependabot to play ball, we might have to drop pnpm and go back to yarn.

@simonbyford simonbyford merged commit 10f0e06 into main Nov 30, 2021
@simonbyford simonbyford deleted the manual-deps-update-30-11-2021 branch November 30, 2021 17:19
simonbyford added a commit that referenced this pull request Dec 1, 2021
We found `pnpm` wasn't compatible with dependabot (see: #436)
@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2021

🎉 This PR is included in version 0.31.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

simonbyford added a commit that referenced this pull request Dec 1, 2021
We found `pnpm` wasn't compatible with dependabot (see: #436)
simonbyford added a commit that referenced this pull request Dec 2, 2021
We found `pnpm` wasn't compatible with dependabot (see: #436)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants