Closed jimsynz closed 3 years ago
Hello, @jimsynz! This is your first Pull Request that will be reviewed by SourceLevel, an automatic Code Review service. It will leave comments on this diff with potential issues and style violations found in the code as you push new commits. You can also see all the issues found on this Pull Request on its review page. Please check our documentation for more information.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
lib/mix/tasks/git_ops.release.ex | 1 | 9 | 11.11% | ||
<!-- | Total: | 1 | 9 | 11.11% | --> |
Totals | |
---|---|
Change from base Build 139: | -0.9% |
Covered Lines: | 240 |
Relevant Lines: | 283 |
LGTM, will merge and release this weekend
🚀 Thank you for your contribution! 🚀
Published as version 2.1.0
This just bypasses the shell prompt so that it can be used non-interactively.
I did not add an acceptance test for this because I can't figure out how to, but am happy to do so with a few pointers.
Contributor checklist
fix: Multiply by appropriate coefficient
, orfeat(Calculator): Correctly preserve history
Any explanation or long form information in your commit message should be in a separate paragraph, separated by a blank line from the primary message