Skip to content
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

Demo_Gen: Improved Error Generation #610

Open
Tracked by #604
ambiguousname opened this issue Jul 30, 2024 · 1 comment
Open
Tracked by #604

Demo_Gen: Improved Error Generation #610

ambiguousname opened this issue Jul 30, 2024 · 1 comment
Labels
B-demo_gen Demo_Gen backend

Comments

@ambiguousname
Copy link
Member

While I was trying to set this up in the ICU4X repo, clearer errors would have been much more useful:

  • I think it'd be good to allow generation even if certain errors (maybe just push warnings instead?) are present, so someone could quickly set up a demo and see how it works.
  • Better warnings/error description would be good. Maybe showing the tree of dependencies, so people know where to look.
@ambiguousname
Copy link
Member Author

I forgot that errors.set_context_method automatically handles some of this for us. Not exactly the stack I want, but we only set this at the start of a method. Which I would say is fairly useful.

What would help even more though is not including javascript in the error messages, because that's going to confuse a lot of people

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B-demo_gen Demo_Gen backend
Projects
None yet
Development

No branches or pull requests

1 participant