Skip to content

Enable Spotless Ratcheting - #120

Merged
kcooney merged 3 commits into
mainfrom
kyle/ratchet-spotless
Jan 21, 2026
Merged

Enable Spotless Ratcheting#120
kcooney merged 3 commits into
mainfrom
kyle/ratchet-spotless

Conversation

@cuttestkittensrule

Copy link
Copy Markdown
Collaborator
  • Requires updating the license year if the file has changed since 'origin/main', and the license year needs to be changed
  • Doesn't require git to be installed
  • Won't format anything not changed since 'origin/main'
    • Fine, since CI ensures 'origin/main' is always fully formatted
  • Requires CI change to fetch origin/main, so spotless can actually have something to compare against

In addition to enabling ratcheting, I updated all of the license years to be accurate.

@cuttestkittensrule

Copy link
Copy Markdown
Collaborator Author

I also did a quick CI test, and it looks like doing a depth 1 clone is sufficient enough to let spotless do it's job. see https://github.com/Prospect-Robotics/lib2813/actions/runs/21199153309 for the log of running the CI when it doesn't have the commit that was branched off of. We could change it to fetch all of main, but that probably isn't necessary if it just works with a depth one fetch

@kcooney kcooney left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sweet!

@kcooney
kcooney merged commit fdd9835 into main Jan 21, 2026
1 check passed
@cuttestkittensrule
cuttestkittensrule deleted the kyle/ratchet-spotless branch January 27, 2026 15:39
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.

2 participants