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

[Bug] Compilation Errors and internal errors are not reported using the logging handler #34

Closed
3 tasks done
Tracked by #43 ...
Luna-Klatzer opened this issue Apr 29, 2022 · 0 comments · Fixed by #43 or #51
Closed
3 tasks done
Tracked by #43 ...
Assignees
Labels
bug Bug or issue in the language or API

Comments

@Luna-Klatzer
Copy link
Member

Is there an existing issue for this?

  • I have searched the existing issues

I am following the documentation's guide

  • I have read the documentation

This issue exists in the latest version

  • I am using the latest version

Current Behavior

When defining the logger parameter of KipperLogger, the compiler only returns DEBUG and INFO messages, but no errors (ERROR) or fatal errors (FATAL).

Expected Behavior

The KipperLogger should receive the traceback using KipperError.getTraceback() and error metadata using the KipperLogger.emitHandler.

Steps To Reproduce

  1. Initialise a new KipperLogger using a defined handler (level, msg) => { ... }.
  2. Run the Compiler in any way that can cause an error.
  3. The exception should only be thrown without any logging message being sent to the logger.

Environment

  • Kipper: v0.3.0
  • Operating System (+ Version): Windows 10 - Build 19043.1645
@Luna-Klatzer Luna-Klatzer added the bug Bug or issue in the language or API label Apr 29, 2022
@Luna-Klatzer Luna-Klatzer added this to the Stable Kipper Release milestone Apr 29, 2022
@Luna-Klatzer Luna-Klatzer self-assigned this Apr 29, 2022
@Luna-Klatzer Luna-Klatzer changed the title [Bug] Errors and internal errors are not reported using the logging handler [Bug] Compilation Errors and internal errors are not reported using the logging handler Apr 29, 2022
@Luna-Klatzer Luna-Klatzer pinned this issue May 1, 2022
Luna-Klatzer added a commit that referenced this issue May 2, 2022
Luna-Klatzer added a commit that referenced this issue May 2, 2022
Luna-Klatzer added a commit that referenced this issue May 2, 2022
Luna-Klatzer added a commit that referenced this issue May 2, 2022
@Luna-Klatzer Luna-Klatzer mentioned this issue May 3, 2022
8 tasks
@Luna-Klatzer Luna-Klatzer unpinned this issue May 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment