You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
After noticing that --no-dev-dependencies did not work, I fixed it in 73e1ac6 and then decided to make it the default in 1ba1e53 .
In the past I thought "well of course as a reviewer you want to know about ALL the dependencies", but from the perspective of time, I think it's making things even more intimidating, since dev crates like criterion etc. bring in a lot of own dependencies and just blow up the dependency tree one has to look through.
And in the end the dev dependencies affect much fewer people.
If anyone wants to verify dev dependencies they can now do cargo crev verify --dev-depencies which is shorter to type and avoids a negation.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
After noticing that
--no-dev-dependencies
did not work, I fixed it in 73e1ac6 and then decided to make it the default in 1ba1e53 .In the past I thought "well of course as a reviewer you want to know about ALL the dependencies", but from the perspective of time, I think it's making things even more intimidating, since dev crates like criterion etc. bring in a lot of own dependencies and just blow up the dependency tree one has to look through.
And in the end the dev dependencies affect much fewer people.
If anyone wants to verify dev dependencies they can now do
cargo crev verify --dev-depencies
which is shorter to type and avoids a negation.Beta Was this translation helpful? Give feedback.
All reactions