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
Add parcel config crate #9700
Add parcel config crate #9700
Conversation
38029f9
to
1a146e5
Compare
}; | ||
} | ||
|
||
test_pipeline_map!(compressors); |
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.
I'd personally write helper functions rather than macro it unless not possible because macros mess-up a lot of IDE features
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.
Yeah I get that, but there's far too much repetition / not enough coverage without the use of macros.
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.
Thanks, the PR looks good.
I left some comments regarding documentation and nits.
1a146e5
to
f2e71bf
Compare
f2e71bf
to
aeae775
Compare
↪️ Pull Request
This change introduces a new
parcel_config
crate that ports some of the logic in v2...core-rs2 and ultimately expands on the implementation there.package_manager
crate and trait to add as a dependency of the config🚨 Test instructions
cargo test -p parcel_config