-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Standardise and clean up Makefile #5025
Conversation
Use check- prefix for test- targets to adhere to Make's standard naming convention [1]. Snake-case names are abandoned in favor of kebab-case naming convention. [1] https://www.gnu.org/prep/standards/html_node/Standard-Targets.html
Codecov Report
@@ Coverage Diff @@
## master #5025 +/- ##
=========================================
+ Coverage 55.5% 55.5% +<.01%
=========================================
Files 287 287
Lines 17538 17540 +2
=========================================
+ Hits 9734 9736 +2
+ Misses 7104 7103 -1
- Partials 700 701 +1 |
.circleci/config.yml
Outdated
@@ -78,35 +78,35 @@ jobs: | |||
executor: golang | |||
steps: | |||
- make: | |||
target: test_sim_nondeterminism | |||
target: test-sim-nondeterminism | |||
description: "Test individual module simulations" | |||
|
|||
test_sim_import_export: |
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.
we should update these too
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.
Fede, if I renamed workflow steps CI would fail. I'd propose to normalise the Makefile targets only in the context of this PR. Does that work for you?
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.
ACK ⚡️
@alessio do you mind fixing CI? |
CI is fixed. Merging is blocked because CircleCI build needs manual intervention as this PR amends the build configuration |
Snake-case names are abandoned in favor of
kebab-case naming convention.
Remove goimports from tools.
Ref: cosmos/gaia#120
docs/
)Unreleased
section inCHANGELOG.md
Files changed
in the github PR explorerFor Admin Use: