In order to be more beginner friendly and to encourage many contributors, don't override the default behavior of cargo #5844
DrakeEvans
started this conversation in
General
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Currently in the cargo.toml, the debug = 0 value is overridden. This is a significant change from the default behavior of cargo. The stated reason is that
"Disabling debug info speeds up builds a bunch,
and we don't rely on it for debugging that much"
I completely understand the reasoning for wanting faster builds but removing the debugging information from the default profile is not the correct way. Make a separate profile for fast builds.
Additionally, the statement "and we don't rely on it for debugging that much" is extremely opinionated and if that is the mantra of the repo then it needs to be documented in the contributing guidelines.
Words cannot express my current outrage. I have spent hours attempting to extend foundry and learn the inner workings which I was unable to do properly because someone who shall not be named decided I didnt need debug info on the default dev profile, yet provided no documentation on this change.
Beta Was this translation helpful? Give feedback.
All reactions