Run 'cargo fmt' to format code #489
No reviewers
Labels
No labels
A: API
A: Backend
A: Federation
A: Front-End
A: I18N
A: Meta
A: Security
Build
C: Bug
C: Discussion
C: Enhancement
C: Feature
Compatibility
Dependency
Design
Documentation
Good first issue
Help welcome
Mobile
Rendering
S: Blocked
S: Duplicate
S: Incomplete
S: Instance specific
S: Invalid
S: Needs Voting/Discussion
S: Ready for review
Suggestion
S: Voted on Loomio
S: Wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: Plume/Plume#489
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "format-using-cargo-fmt"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Codecov Report
Hey, thank you! Would you want to try to add it to the CI too? (it's fine if you don't, we can totally do that in another PR)
Hi @BaptisteGelez I would like to add it. I am new to Plume project. Can you please point me to where in the .travis.yml file can
cargo fmt
be added?You can probably add a new item at the end of the
jobs
section, in the "test and coverage" stage (I'm not sure if this indication is clear enough, tell me if it is not 😅)given that the usual way to run
rustfmt
is in "check" mode, we can probably run it as first job. and from what i gather,rustfmt
doesn't care about features, and just formats all code? or? did it format the test code, too?If the code isn't formatted ('rustfmt' will exit with 1 if there is an error in formatting - https://github.com/rust-lang/rustfmt#running) then is it still ok to go to the build stage?
Suggestion: We can add a
check
stage which will run before the build stage.@BaptisteGelez @igalic What do you think?
@BaptisteGelez @igalic I tried adding cargo fmt but the CI fails for weird reason.
I tried locally but I didn't get this error - https://travis-ci.org/Plume-org/Plume/jobs/508992511#L584 & nor did it fail for the build before adding
cargo fmt
to travis.yml.I tried to fix that but the build fails - https://travis-ci.org/Plume-org/Plume/jobs/509005663#L589
The build for this PR had passed before adding
cargo fmt
to travis build stages - https://travis-ci.org/Plume-org/Plume/builds/508519598I suggest we can add
cargo fmt
to CI in another PR.OK, no problem. I'll merge this as soon as you'll have reverted the changes to
.travis.yml
. And thank you again for your work. 🙂@BaptisteGelez done. I have updated the PR.