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

chore: add script to promote rules #4419

Merged
merged 1 commit into from
Apr 28, 2023
Merged

Conversation

ematipico
Copy link
Contributor

Summary

This PR creates a new just script to automate the process of promoting the rules:

just promote-rule noConsoleLog style

Test Plan

Changelog

  • The PR requires a changelog line

Documentation

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

@netlify
Copy link

netlify bot commented Apr 28, 2023

Deploy Preview for docs-rometools canceled.

Name Link
🔨 Latest commit 0cc7bc5
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/644baf16e5cb1d00088f67f2

@github-actions github-actions bot added A-Linter Area: linter A-Tooling Area: our own build, development, and release tooling labels Apr 28, 2023
cargo run -p xtask_codegen -- promoterule --rule={{rule}} --group={{group}}
just codegen-linter
just documentation
-cargo test -p rome_js_analyze -- {{snakecase(rule)}}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This - tells just to continue the execution of the scripts even if there's an error

@ematipico ematipico force-pushed the chore/script-to-promote-rule branch from 9167f25 to 254ec2c Compare April 28, 2023 10:50
@github-actions
Copy link

Parser conformance results on ubuntu-latest

js/262

Test result main count This PR count Difference
Total 48863 48863 0
Passed 47796 47796 0
Failed 1067 1067 0
Panics 0 0 0
Coverage 97.82% 97.82% 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 6212 6212 0
Passed 1763 1763 0
Failed 4449 4449 0
Panics 0 0 0
Coverage 28.38% 28.38% 0.00%

ts/babel

Test result main count This PR count Difference
Total 639 639 0
Passed 573 573 0
Failed 66 66 0
Panics 0 0 0
Coverage 89.67% 89.67% 0.00%

ts/microsoft

Test result main count This PR count Difference
Total 17224 17224 0
Passed 13122 13122 0
Failed 4102 4102 0
Panics 0 0 0
Coverage 76.18% 76.18% 0.00%

@ematipico ematipico force-pushed the chore/script-to-promote-rule branch from 254ec2c to e74ca5d Compare April 28, 2023 11:30
@ematipico ematipico force-pushed the chore/script-to-promote-rule branch from e74ca5d to 0cc7bc5 Compare April 28, 2023 11:33
@ematipico ematipico merged commit a98f7d1 into main Apr 28, 2023
@ematipico ematipico deleted the chore/script-to-promote-rule branch April 28, 2023 11:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Linter Area: linter A-Tooling Area: our own build, development, and release tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant