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
chore: Add sbt command for python stylechecking #2156
base: master
Are you sure you want to change the base?
Conversation
Acrolinx ScorecardsA minimum Acrolinx score of 80 is required. Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.
More information about Acrolinx |
Hey @BrendanWalsh 👋! We use semantic commit messages to streamline the release process. Examples of commit messages with semantic prefixes:
To test your commit locally, please follow our guild on building from source. |
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 missing var, other than that all good!
Acrolinx ScorecardsA minimum Acrolinx score of 80 is required. Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.
More information about Acrolinx |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2156 +/- ##
==========================================
- Coverage 86.35% 86.26% -0.09%
==========================================
Files 325 325
Lines 16959 16959
Branches 1524 1524
==========================================
- Hits 14645 14630 -15
- Misses 2314 2329 +15 ☔ View full report in Codecov by Sentry. |
Acrolinx ScorecardsA minimum Acrolinx score of 80 is required. Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.
More information about Acrolinx |
Acrolinx ScorecardsA minimum Acrolinx score of 80 is required. Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.
More information about Acrolinx |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Acrolinx ScorecardsA minimum Acrolinx score of 80 is required. Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.
More information about Acrolinx |
Azure Pipelines successfully started running 1 pipeline(s). |
The build enforces python style but there is no convenient way to test locally without investigating the build pipeline to see how it tests
This introduces an sbt command so that you can easily check python style locally in a way that is consistent with the build