Skip to content

update eslint to v10 and fix linting issues - #636

Open
cecilialau6776 wants to merge 2 commits into
ComputerScienceHouse:developfrom
cecilialau6776:eslint-v10
Open

cecilialau6776 wants to merge 2 commits into
ComputerScienceHouse:developfrom
cecilialau6776:eslint-v10

Conversation

@cecilialau6776

@cecilialau6776 cecilialau6776 commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

What

Javascript

  • Remove eslint-config-google
  • Add eslint: "^10.12.0"
  • Add gts
  • Add lint and fix npm scripts

Misc

  • Update readme to link OIDC dev creds to the CSH wiki
  • Add GitHub CI/CD action to lint with eslint

Why

Conditional needs a Javascript linter.

I installed GTS as a replacement for eslint-config-google.

Test Plan

N/A

Env Vars

N/A

Documentation

N/A

Checklist

  • Tested all changes locally

@datadog-computersciencehouse

datadog-computersciencehouse Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Pipelines

⚠️ Warnings

Your PR has warnings. Please review the issues below.

🚦 1 Pipeline job failed

Sonarqube | Sonarqube Analysis

View in Datadog · View in GitHub Actions

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 6a51520 | Docs | View more details | Give us feedback!

@cecilialau6776
cecilialau6776 marked this pull request as ready for review October 7, 2026 17:39

This branch has not been deployed

No deployments
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.

1 participant