-
Notifications
You must be signed in to change notification settings - Fork 28
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
Apply and enforce formatting #155
Conversation
670a6ef
to
de60386
Compare
rebased on master |
de60386
to
0ed6359
Compare
rebase and tidy commits |
github action run results can be checked here - https://github.com/danieleades/upgrade/pull/3/checks?check_run_id=2360001747 |
0ed6359
to
1fe6d81
Compare
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.
No regressions detected.
Formatting changes will destroy mergeability of development branch. Had to undo the recently-merged commits. |
@mmstick I take it you don't want to persue enforcing the formatting policy in CI? |
Maybe when it's not actively in development |
Fair enough. Though I would point out that a pull request to add formatting is, by it's very nature, very easy to rebase. |
apply formatting, and add a CI action to check formatting
closes #152