-
Notifications
You must be signed in to change notification settings - Fork 636
multi: Introduce fmt-check
to CI workflow
#1042
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
multi: Introduce fmt-check
to CI workflow
#1042
Conversation
make fmt-check
to CI workflowfmt-check
to CI workflow
cd6161f
to
bd50d85
Compare
747f627
to
582483d
Compare
651f149
to
bc3a736
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.
tACK
Co-authored-by: Gustavo Stingelin <59903150+GustavoStingelin@users.noreply.github.com> Co-authored-by: Oliver Gugger <gugger@gmail.com>
bc3a736
to
c931ade
Compare
c931ade
to
3b9f539
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.
One small comment, otherwise LGTM 🎉
3b9f539
to
4abc517
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.
Thank you!!
Change Description
Towards #1041.
Towards #1015.
Steps to Test
Steps for reviewers to follow to test the change.
Pull Request Checklist
Testing
Code Style and Documentation
📝 Please see our Contribution Guidelines for further guidance.