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

Renovate fails to update dependencies correctly #1252

Open
MikeMcC399 opened this issue Sep 7, 2024 · 4 comments
Open

Renovate fails to update dependencies correctly #1252

MikeMcC399 opened this issue Sep 7, 2024 · 4 comments
Labels
bug Something isn't working ci type: dependencies

Comments

@MikeMcC399
Copy link
Collaborator

Issue

When Renovate attempts to update dependencies from package.json it regularly fails the CI workflow .github/workflows/check-dist.yml which runs

npm ci
npm run format
npm run build

and checks that there are no uncommitted changes.

Examples of failed Renovate PRs, attempting to update dependencies, are:

Renovate is not re-formatting and re-building the app.

Suggestion

Renovate should either be reconfigured to correctly reformat and rebuild the action, or it should avoid attempts to update dependencies and restrict itself to devDependencies only.

@MikeMcC399 MikeMcC399 added bug Something isn't working type: dependencies ci labels Sep 7, 2024
@MikeMcC399
Copy link
Collaborator Author

@MikeMcC399
Copy link
Collaborator Author

This is a call to the Cypress.io team as the server-side of the Renovate configuration is not publicly accessible, so it makes more sense to have any adjustments to the Renovate configuration also handled internally.

@jennifer-shehane
Copy link
Member

@MikeMcC399 Can this not be updated in the renovate.json file?

@MikeMcC399
Copy link
Collaborator Author

MikeMcC399 commented Dec 30, 2024

@jennifer-shehane

Can this not be updated in the renovate.json file?

Apparently Renovate does not support running pre-commit hooks, such as from Husky, so it seems that the answer to your question is: no.

It would be possible to configure renovate so that it does not attempt to update npm dependencies, which would avoid unsuccessful PRs being created. On the other hand, there is some value to being alerted to new versions by Renovate, even if the auto-created PRs are invalid.

In each case the options are then:

  • manually (re-)build the action in the Renovate branch and add a new commit
  • close the Renovate PR and replace it with a manual PR
  • close the Renovate PR and skip the update

I did some spot-checks on other repos containing GitHub Actions and I couldn't find any repos where this issue has been resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ci type: dependencies
Projects
None yet
Development

No branches or pull requests

2 participants