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

refactor(telemetry-generator): Upgrade eslint config to "recommended" and fix violations in code #21076

Merged

Conversation

Josmithr
Copy link
Contributor

@Josmithr Josmithr commented May 14, 2024

This package was using the minimal-deprecated configuration, which is scheduled for deletion. Upgrades the package's config to the recommended base and fixes violations in the code.

This PR disables some of the recommended rules due to the sheer number of violations. These can and should be removed and fixed in the future. For now, the primary goal here is to remove usages of the deprecated config.

AB#2900

@Josmithr Josmithr requested review from alexvy86, WayneFerrao and a team May 14, 2024 20:26
@github-actions github-actions bot added the base: main PRs targeted against main branch label May 14, 2024
Copy link
Contributor

@alexvy86 alexvy86 left a comment

Choose a reason for hiding this comment

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

One required change below, otherwise lgtm

tools/telemetry-generator/src/handlers/testPassRate.ts Outdated Show resolved Hide resolved
@Josmithr Josmithr enabled auto-merge (squash) May 14, 2024 21:06
@Josmithr Josmithr merged commit 0793037 into microsoft:main May 14, 2024
26 checks passed
@Josmithr Josmithr deleted the telemetry-generator/promote-eslint-config branch May 14, 2024 21:18
kekachmar pushed a commit to kekachmar/FluidFramework that referenced this pull request May 21, 2024
… and fix violations in code (microsoft#21076)

This package was using the `minimal-deprecated` configuration, which is
scheduled for deletion. Upgrades the package's config to the
`recommended` base and fixes violations in the code.

This PR disables some of the `recommended` rules due to the sheer number
of violations. These can and should be removed and fixed in the future.
For now, the primary goal here is to remove usages of the deprecated
config.


[AB#2900](https://dev.azure.com/fluidframework/235294da-091d-4c29-84fc-cdfc3d90890b/_workitems/edit/2900)

---------

Co-authored-by: Alex Villarreal <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
base: main PRs targeted against main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants