-
Notifications
You must be signed in to change notification settings - Fork 105
Add Renovate and Prettier configurations and CI script #143
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Also, a heads up, it seems that there are some conflicts that need to be solved @kenjonespizza :) |
RitaDias
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are some conflicts but the code itself looks good!
|
Thanks so much, @RitaDias ! I merged in |
Description
I've added a CI pipeline for linting, dependency management (with Renovate), and automated PR creation for formatting code with Prettier. A lot of this setup is highly inspired/taken from Cody and teams work on other templates like https://github.com/sanity-io/template-nextjs-personal-website
What to review
This is my first rodeo working on this type of upgrade. It'd be great if someone with experience can take a looksy and give thoughts or approval of the set.
Things to look at:
Testing
I added this branch to trigger the CI and Prettier workflows. Assuming those run correctly, Testing shoudn't be too hard 😅