8000 feat: add eslint for frontend project by tjsilver · Pull Request #558 · guardian/janus-app · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: add eslint for frontend project #558

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 3 commits into from
Apr 28, 2025
Merged

Conversation

tjsilver
Copy link
Contributor
@tjsilver tjsilver commented Apr 24, 2025

What is the purpose of this change?

What is the value of this change and how do we measure success?

  • We are applying best practices to our frontend code by ensuring the files are linted.

Next steps

  • Add linting to CI.

  • Tested locally

@tjsilver tjsilver force-pushed the ts/add-frontend-linting branch from 653b7bf to 131e7d2 Compare April 28, 2025 14:11
Copy link
Member
@kelvin-chappell kelvin-chappell left a comment

Choose a reason for hiding this comment

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

👍

Fix: linting errors identified by eslint
@tjsilver tjsilver merged commit 2d4cb8d into main Apr 28, 2025
6 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@tjsilver tjsilver deleted the ts/add-frontend-linting branch April 28, 2025 15:03
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
0