Skip to content

Added logging module#42

Open
cornusandu wants to merge 4 commits intoNotValra:mainfrom
cornusandu:patch-2
Open

Added logging module#42
cornusandu wants to merge 4 commits intoNotValra:mainfrom
cornusandu:patch-2

Conversation

@cornusandu
Copy link

Added a separate logging module. This allows for:

  • disabling low-priority logs
  • disabling alert() calls (for testing)
  • console messages formatted with built-in timestamps and log levels
  • dispatching between console.debug(), console.info(), console.warn(), and console.error() instead of just console.log() and console.error()

cornusandu and others added 2 commits March 12, 2026 21:28
Signed-off-by: Bogdan Sandu <bogdanelsandu2011@gmail.com>
@cornusandu
Copy link
Author

I've dealt with the merge conflicts. This is ready to merge.

Please tell me if you'd like any modifications to be made.

cornusandu and others added 2 commits March 12, 2026 21:45
Signed-off-by: Bogdan Sandu <bogdanelsandu2011@gmail.com>
@NotValra
Copy link
Owner

Working on a bigger change to all scripts for locale. So i recommend waiting until thats done and then fixing the conflicts

@cornusandu
Copy link
Author

Working on a bigger change to all scripts for locale. So i recommend waiting until thats done and then fixing the conflicts

Alright. I would appreciate it if you could let me know if you'll merge this PR, though, so I know whether to later fix the conflicts or leave it as-is.

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

Successfully merging this pull request may close these issues.

2 participants