Skip to content

feat: eslint rules set 1#172

Open
Alessandro100 wants to merge 1 commit into
mainfrom
feat/162-eslint-set1
Open

feat: eslint rules set 1#172
Alessandro100 wants to merge 1 commit into
mainfrom
feat/162-eslint-set1

Conversation

@Alessandro100

Copy link
Copy Markdown
Contributor

Summary:

closes #162

Enables the Eslint rules

      '@typescript-eslint/no-unnecessary-type-assertion': 'off',
      '@typescript-eslint/no-redundant-type-constituents': 'off',
      '@typescript-eslint/require-await': 'off',

Expected behavior:

The application should behave the exact same way with cleaner coding practices

Testing tips:

Running yarn lint should pass
In this PR the features affected the most would be

  • map precompute -> to go the mbta detali map and assure it still works
  • The add feed form (basic testing)

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Run the unit tests with yarn test to make sure you didn't break anything
  • Add or update any needed documentation to the repo
  • Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
  • Linked all relevant issues
  • Include screenshot(s) showing how this pull request works and fixes the issue(s)

@Alessandro100 Alessandro100 self-assigned this Jul 17, 2026
@vercel

vercel Bot commented Jul 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
mobilitydatabase-web Ready Ready Preview, Comment Jul 17, 2026 2:39pm

Request Review

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.

Enable eslint rules set 1

1 participant