ci: split up CI jobs #52
Conversation
robjtede
force-pushed
the
ci
branch
5 times, most recently
from
4413d2b to
cf27abb
Compare
| # fd --hidden -e=toml --exec-batch taplo format --check | ||
| # fd --hidden -e=toml --exec-batch taplo lint | ||
| # fd --hidden --type=file -e=md -e=yml --exec-batch prettier --check |
There was a problem hiding this comment.
using taplo instead of cargo-sort is my preference but leaving it out for now
also using prettier for markdown and YAML formatting keeps things consistent, but also leaving it out for now and just commenting it here for future reference
README.md
Outdated
| ```toml | ||
| [dependencies] | ||
| bytesize = { version = "2", features = ["serde"]} | ||
| ``` |
There was a problem hiding this comment.
it's not really useful to place this in crate readmes any more since all crates.io pages suggests cargo add already
There was a problem hiding this comment.
@MrCroxx much of this CI setup is inspired by my other projects, happy to take questions about motivation for any changes
for example: only running tests in this file (used for the readme badge) prevents non-critical failures like formatting from surfacing there
|
Good job! Let me review it, it might takes 1-3 days because I'm not familiar with |
|
BTW. @robjtede How do you like |
|
|
|
Maybe we can release a new version after this PR. 🥰 Or after cleaning some feature requests! |
No description provided.