Skip to content

Integrate js-lint and fix up any newfound errors #413

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

Merged
merged 2 commits into from
May 20, 2025

Conversation

CDeltakai
Copy link
Contributor

@CDeltakai CDeltakai commented May 20, 2025

Description

This PR integrates the new js-lint package which will replace current linting systems. It will also fix up any previously undiscovered errors found by the new linter.

Issues Fixed

Tasks

  • 1. Integrate js-lint and remove old linting elements
  • 2. Fix up any linting errors found by the new linter

Final checklist

  • Domain specific tests
  • Full tests
  • Updated inline-comment documentation
  • Lint fixed
  • Squash and rebased
  • Sanity check the final build

@CDeltakai CDeltakai self-assigned this May 20, 2025
@CDeltakai CDeltakai merged commit ea69b5f into staging May 20, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Integrate js-lint
1 participant