-
Notifications
You must be signed in to change notification settings - Fork 651
Open
Labels
😤 annoyingSomething in the UI / SDK is annoying to useSomething in the UI / SDK is annoying to use🧑💻 dev experiencedeveloper experience (excluding CI)developer experience (excluding CI)
Description
Describe the annoyance
Originally made as an TODO by @emilk:
| #[allow(clippy::unnecessary_wraps)] // TODO(emilk): implement proper error handling instead of exiting on the first cargo_error |
The build scripts are more mature now with cargo_error from #10941, issues are reported one by one which may degrade developer experience if something goes wrong and it requires a several iterations of fixes and compilations.
Additional context
As not everything in crates/build is for build scripts, some panic! should be changed to use the reporter passed in.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
😤 annoyingSomething in the UI / SDK is annoying to useSomething in the UI / SDK is annoying to use🧑💻 dev experiencedeveloper experience (excluding CI)developer experience (excluding CI)