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

Change unkown regions/variables/... to throw proper errors #434

Open
phackstock opened this issue Dec 5, 2024 · 4 comments
Open

Change unkown regions/variables/... to throw proper errors #434

phackstock opened this issue Dec 5, 2024 · 4 comments

Comments

@phackstock
Copy link
Contributor

No description provided.

@danielhuppmann
Copy link
Member

I guess something like the pydantic error collection would be nice...

@phackstock
Copy link
Contributor Author

If only someone wrote something like that 😆

@danielhuppmann
Copy link
Member

I mean, an easy way would be to just write all error messages to a string and then include the string in the error message?

@phackstock
Copy link
Contributor Author

Sure, it's going to be a short issue to tackle. The longest part is probably going to be to adjust the tests.
Potentially, we should align the entire error reporting. There are a number of other checks that rely on writing to the log instead of putting all the information into the error message.

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

No branches or pull requests

2 participants