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

Add self-hosted renovate #80193

Merged
merged 27 commits into from
Aug 8, 2023
Merged

Add self-hosted renovate #80193

merged 27 commits into from
Aug 8, 2023

Conversation

noahtallen
Copy link
Contributor

@noahtallen noahtallen commented Aug 3, 2023

Some time ago, Renovate was suspended at the organization level due to a security issue. While that was fixed, the account remains suspended. This PR proposes moving our config to "self-hosted" via GitHub actions, inspired by Jetpack. (Automattic/jetpack#23307)

I copy/pasted the config file from Jetpack (https://github.com/Automattic/jetpack/blob/trunk/.github/workflows/renovate.yml), and added the same RENOVATE_TOKEN secret to wp-calypso. (See pMz3w-eQ7-p2#comment-96792) I did make some small changes as well.

Testing:

Check github workflow run!

image

@noahtallen noahtallen requested review from a team August 3, 2023 20:25
@noahtallen noahtallen requested a review from a team as a code owner August 3, 2023 20:25
@noahtallen noahtallen self-assigned this Aug 3, 2023
@matticbot matticbot added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Aug 3, 2023
@matticbot
Copy link
Contributor

matticbot commented Aug 3, 2023

@matticbot
Copy link
Contributor

This PR does not affect the size of JS and CSS bundles shipped to the user's browser.

Generated by performance advisor bot at iscalypsofastyet.com.

@noahtallen
Copy link
Contributor Author

See the new dashboard: #80201

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved this to json5 because comments are super helpful to have

Comment on lines +25 to +31
groupName: 'typescript-related packages',
matchPackagePatterns: [ 'typescript-eslint' ],
matchPackageNames: [ 'typescript' ],
prPriority: 2,
Copy link
Contributor

Choose a reason for hiding this comment

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

Question (non-blocking): Mostly out of curiosity, why don't we want @types packages included in the renovate update PRs? Have we had issues with them getting too far ahead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We do want @types to update! It just doesn't make sense to update them with Typescript -- they should be updated when the corresponding package is updated. For example, @types/react probably only needs to be updated when react is updated.

By the way, the config automatically updates everything by default when we don't specify a certain package (unless we manually ignore it). The groups we set up just add our own own settings on top of that, to help us be more organized.

@dpasque
Copy link
Contributor

dpasque commented Aug 4, 2023

Overall, this change looks good to me! 😄 I haven't done too much with renovate bot in the past, so I might be missing some context about some of the package update rules, and it might be good to have someone from frameworks double-check.

Thanks for doing this, that's awesome that we'll get renovate running again! 🥳

@noahtallen noahtallen force-pushed the try-renovate-github-actions branch from 8e06823 to ca9e55f Compare August 7, 2023 22:53
Copy link
Member

@tyxla tyxla left a comment

Choose a reason for hiding this comment

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

Awesome to see this! Left a few minor suggestions, but I think it's good to go!

Let's give it a shot 🚀

renovate.json5 Outdated Show resolved Hide resolved
renovate.json5 Outdated Show resolved Hide resolved
noahtallen and others added 2 commits August 8, 2023 12:53
@noahtallen
Copy link
Contributor Author

Great suggestions; added!

@noahtallen noahtallen merged commit 0895c51 into trunk Aug 8, 2023
@noahtallen noahtallen deleted the try-renovate-github-actions branch August 8, 2023 20:08
@github-actions github-actions bot removed the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants