Skip to content

Update backend to eslint v9.x#413

Merged
qhanson55 merged 8 commits intomasterfrom
feature/eslint
Jan 19, 2026
Merged

Update backend to eslint v9.x#413
qhanson55 merged 8 commits intomasterfrom
feature/eslint

Conversation

@kyle1morel
Copy link
Collaborator

@kyle1morel kyle1morel commented Jan 13, 2026

Description

Backend changes
Update tsconfig to correctly extend from @tsconfig/node24/tsconfig.json
Update eslint to v9.x
Currently using tseslint.configs.recommended & tseslint.configs.stylistic instead of the TypeChecked variants. Recommend moving to those soon, but additional (difficult) changes will be required.
Thousands of lint changes to comply with new eslint v9.x settings.

Frontend changes
Array -> [] syntax. This slipped in during a replace all. Will be required anyways once the frontend eslint is updated as well.

https://apps.nrs.gov.bc.ca/int/jira/browse/PADS-316

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

@github-actions
Copy link

github-actions bot commented Jan 13, 2026

Coverage Report (Application)

Totals Coverage
Statements: 81.32% ( 3409 / 4192 )
Methods: 77.1% ( 431 / 559 )
Lines: 86.8% ( 2414 / 2781 )
Branches: 66.2% ( 564 / 852 )

@github-actions
Copy link

github-actions bot commented Jan 13, 2026

Coverage Report (Frontend)

Totals Coverage
Statements: 45.21% ( 4649 / 10283 )
Methods: 31.26% ( 527 / 1686 )
Lines: 55.72% ( 3017 / 5415 )
Branches: 34.73% ( 1105 / 3182 )

@kyle1morel kyle1morel changed the title Feature/eslint Update backend to eslint v9.x Jan 13, 2026
@kyle1morel kyle1morel marked this pull request as ready for review January 13, 2026 19:57
@kyle1morel kyle1morel requested a review from a team as a code owner January 13, 2026 19:57
@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed for 'nr-permitconnect-navigator-service app'

Issues
19 New issues
0 Accepted issues

Measures
0 Security Hotspots
84.4% Coverage on New Code
1.6% Duplication on New Code

See analysis details on SonarQube Cloud

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed for 'nr-permitconnect-navigator-service frontend'

Failed conditions
40.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@qhanson55 qhanson55 merged commit 7a054a5 into master Jan 19, 2026
30 of 31 checks passed
@qhanson55 qhanson55 deleted the feature/eslint branch January 19, 2026 17:39
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