Skip to content

fix: remove unused errors variable in error-logger.js - #7549

Merged
janavipandole merged 1 commit into
janavipandole:mainfrom
tmdeveloper007:fix/#7516-error-logger-dead-code-clean
Aug 13, 2026
Merged

fix: remove unused errors variable in error-logger.js#7549
janavipandole merged 1 commit into
janavipandole:mainfrom
tmdeveloper007:fix/#7516-error-logger-dead-code-clean

Conversation

@tmdeveloper007

Copy link
Copy Markdown
Contributor

📄 Description

Removed the unused let errors = [] declaration that was immediately overwritten in the next statement, eliminating dead code in error-logger.js.

🔗 Related Issues

Closes #7516

🧩 Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Style / UI improvement
  • Refactor
  • Test
  • Chore / dependency update

✅ Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Note: This task is being handled by tmdeveloper007 as part of GSSOC (GirlScript Summer of Code) — please assign this PR to the tmdeveloper007 account when picking it up.

@tmdeveloper007 tmdeveloper007 changed the title fix : remove unused errors variable in error-logger.js fix: remove unused errors variable in error-logger.js Aug 12, 2026
@janavipandole
janavipandole merged commit a33289f into janavipandole:main Aug 13, 2026
21 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix : remove unused errors variable in error-logger.js

2 participants