Skip to content

Conversation

@kenjonespizza
Copy link
Collaborator

@kenjonespizza kenjonespizza commented Aug 6, 2025

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:

  • All the .github/workflow files
  • .github/renovate.json

Testing

I added this branch to trigger the CI and Prettier workflows. Assuming those run correctly, Testing shoudn't be too hard 😅

@vercel
Copy link

vercel bot commented Aug 6, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
template-nextjs-clean Ready Ready Preview Comment Aug 18, 2025 2:13pm

@socket-security
Copy link

socket-security bot commented Aug 6, 2025

@kenjonespizza kenjonespizza requested a review from kmelve August 6, 2025 16:36
@kmelve kmelve removed their request for review August 6, 2025 16:41
@kenjonespizza kenjonespizza marked this pull request as ready for review August 12, 2025 22:08
@kenjonespizza kenjonespizza requested a review from a team as a code owner August 12, 2025 22:08
@RitaDias
Copy link

Also, a heads up, it seems that there are some conflicts that need to be solved @kenjonespizza :)

RitaDias
RitaDias previously approved these changes Aug 18, 2025
Copy link

@RitaDias RitaDias left a 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!

@kenjonespizza
Copy link
Collaborator Author

Thanks so much, @RitaDias ! I merged in main and made one change (removed unnecessary @sanity/template-validator dependency from package). Ready for re-review :)

@kenjonespizza kenjonespizza requested a review from RitaDias August 18, 2025 15:20
@kenjonespizza kenjonespizza merged commit f130b40 into main Aug 18, 2025
8 checks passed
@kenjonespizza kenjonespizza deleted the feat/ci-renovate-prettier branch August 18, 2025 15:49
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 19, 2025
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