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

Support error reporting for required modules #28

Open
samdeane opened this issue Sep 15, 2021 · 5 comments
Open

Support error reporting for required modules #28

samdeane opened this issue Sep 15, 2021 · 5 comments
Labels
enhancement New feature or request NQ-COMMENTED This issue or pull request already exists

Comments

@samdeane
Copy link

If an error occurs in a module you've imported with require, the current reporting system just gives up.

Instead it should be possible to report the error, module and line number, as usual.

Whilst using require is obviously problematic for production, it's immensely useful during development.

@EasternGamer
Copy link

Might be a limitation, but maybe they're not really aware of it.

@EasternGamer
Copy link

@Archaegeo, was there something like exCall or expCall? I can't remember.

@samdeane
Copy link
Author

You can use pcall to trap errors, but you don't get back everything you need, and you have to do it manually anyway, which is daft.

@Archaegeo Archaegeo added the enhancement New feature or request label Sep 20, 2021
@NQ-Ligo
Copy link

NQ-Ligo commented May 3, 2022

Hi there !

Improved error tracking is on our list, but we can't give a date or promise; many priority items are on the list long before that.

@Archaegeo Archaegeo added the NQ-COMMENTED This issue or pull request already exists label May 3, 2022
@samdeane
Copy link
Author

samdeane commented May 4, 2022

many priority items are on the list long before that.

I understand why that might be, but honestly, do everyone a favour. Fix the tools before you fix the API. You will be using the tools yourself to test the changes you make to the API.

And if your management won't sign off on it because they don't get that, then send 'em to me and I'll knock their heads together ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request NQ-COMMENTED This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

4 participants