Skip to content
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

[MAINTENANCE] Fix all Integrations lints. #1363

Open
1 of 4 tasks
Swiddis opened this issue Jan 16, 2024 · 1 comment
Open
1 of 4 tasks

[MAINTENANCE] Fix all Integrations lints. #1363

Swiddis opened this issue Jan 16, 2024 · 1 comment
Assignees
Labels
integrations Used to denote items related to the Integrations project maintenance

Comments

@Swiddis
Copy link
Collaborator

Swiddis commented Jan 16, 2024

Is your feature request related to a problem?
Since the new Lint action was introduced by Eric, there are lots of lint issues lying dormant in Integrations. This is a catchall issue for fixing these "zombie lints", as indicated in e.g. this PR feedback.

What solution would you like?
There are four parts to the integration codebase that can be fixed together or separately. The majority of them should be autofixable, but some will require some manual tweaking:

  • Adaptor Backend server/adaptors/integrations
  • Routing server/routes/integrations
  • React Frontend public/components/integrations
  • Cypress Tests .cypress/integration/integrations_test

What alternatives have you considered?
N/A

Do you have any additional context?
N/A

@Swiddis Swiddis self-assigned this Jan 16, 2024
@Swiddis Swiddis added integrations Used to denote items related to the Integrations project and removed untriaged labels Jan 16, 2024
@Swiddis
Copy link
Collaborator Author

Swiddis commented Jan 26, 2024

Routing lints covered as part of #1381

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integrations Used to denote items related to the Integrations project maintenance
Projects
Status: No status
Development

No branches or pull requests

1 participant