Skip to content

Conversation

@loveboat
Copy link
Contributor

@loveboat loveboat commented Dec 3, 2025

Ticket: PAE-502

Description

eslint no unused vars - this came up whilst I was doing the /dev endpoint. seems we were missing a few of these


Please see the Pull Requests standards.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 4, 2025

Copy link
Contributor

@carlesandres carlesandres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. I'm aware we are not supposed to write our own eslint rules but the current config is getting in the way of our deadlines while providing very little value.

I think we can always consider alternatives to the custom linting rule in the future if there's appetite.

@loveboat
Copy link
Contributor Author

loveboat commented Dec 4, 2025

lgtm. I'm aware we are not supposed to write our own eslint rules but the current config is getting in the way of our deadlines while providing very little value.

I think we can always consider alternatives to the custom linting rule in the future if there's appetite.

I added this mainly to catch a bunch of unused vars that we were leaving dangling

@loveboat loveboat merged commit 3136bbd into main Dec 4, 2025
7 checks passed
@loveboat loveboat deleted the PAE-502-eslint-no-unused-vars branch December 4, 2025 10:04
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.

3 participants