Skip to content

Implement Logger system #1020

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 1, 2024
Merged

Implement Logger system #1020

merged 2 commits into from
Sep 1, 2024

Conversation

@bajrangCoder
Copy link
Collaborator

@bajrangCoder bajrangCoder commented Aug 29, 2024 •
edited
Loading

  • Logger system
  • Removed some old console.log and console.error with logger api
  • Added new log method to window object
  • Now avoid using console logs instead use logger , so it will be easy to debug any bug
  • Logging system is efficient and it automatically manages flushing of buffer and LRU algorithm to keep log file small and relevant
  • Search for "Open Log file" in command palette to open the log file in editor.

@bajrangCoder
Implemented Logger system
4425bb7 
- Logger system
- Removed some old `console.log` and `console.error` with logger api
- Added new log method to `window` object
@bajrangCoder bajrangCoder changed the title Implemented Logger system Implement Logger system Aug 29, 2024
UnschooledGamer
UnschooledGamer requested changes Aug 29, 2024
View reviewed changes
@UnschooledGamer
Copy link
Member

UnschooledGamer commented Aug 29, 2024

By-mistake Selected the Wrong Review Type. Sorry.

@bajrangCoder bajrangCoder added the enhancement New feature or request label Aug 29, 2024
@bajrangCoder
few improvement and fixes
37fea48 
- Added a option to open log file in editor from Command Palette
- Few fixes
@bajrangCoder bajrangCoder merged commit 389bb69 into Acode-Foundation:main Sep 1, 2024
@bajrangCoder bajrangCoder deleted the logging-system branch December 19, 2024 06:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@UnschooledGamer UnschooledGamer UnschooledGamer requested changes

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bajrangCoder @UnschooledGamer