Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

chore(codegen): fix codegen for new lint rule #4057

Merged
merged 2 commits into from
Dec 15, 2022
Merged

Conversation

xunilrj
Copy link
Contributor

@xunilrj xunilrj commented Dec 15, 2022

Summary

This PR fixes codegen for new lint rules.

Test Plan

> just new-lintrule /crates/rome_js_analyze/src/semantic_analyzers/nursery SomeRule

Documentation

  • The PR requires documentation
  • I will create a new PR to update the documentation

@xunilrj xunilrj requested a review from a team as a code owner December 15, 2022 11:32
@netlify
Copy link

netlify bot commented Dec 15, 2022

Deploy Preview for docs-rometools canceled.

Name Link
🔨 Latest commit 1207134
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/639b075249b6f2000955716e

@github-actions
Copy link

Parser conformance results on ubuntu-latest

js/262

Test result main count This PR count Difference
Total 48647 48647 0
Passed 47582 47582 0
Failed 1065 1065 0
Panics 0 0 0
Coverage 97.81% 97.81% 0.00%

jsx/babel

Test result main count This PR count Difference
Total 40 40 0
Passed 37 37 0
Failed 3 3 0
Panics 0 0 0
Coverage 92.50% 92.50% 0.00%

symbols/microsoft

Test result main count This PR count Difference
Total 6093 6093 0
Passed 1754 1754 0
Failed 4339 4339 0
Panics 0 0 0
Coverage 28.79% 28.79% 0.00%

ts/babel

Test result main count This PR count Difference
Total 639 639 0
Passed 562 562 0
Failed 77 77 0
Panics 0 0 0
Coverage 87.95% 87.95% 0.00%

ts/microsoft

Test result main count This PR count Difference
Total 16740 16740 0
Passed 12814 12814 0
Failed 3926 3926 0
Panics 0 0 0
Coverage 76.55% 76.55% 0.00%

@xunilrj xunilrj merged commit 2bf8df5 into main Dec 15, 2022
@xunilrj xunilrj deleted the fix/lint-rule-codegen branch December 15, 2022 12:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants