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

feat(rome_js_parser): Implement option-passing mechanism for parser test #4631 #4652

Merged
merged 1 commit into from
Jul 5, 2023

Conversation

denbezrukov
Copy link
Contributor

@denbezrukov denbezrukov commented Jul 3, 2023

Summary

Close #4631

Example: test test_type test_name {parse_class_parameter_decorators: true/false }

We need to parse JS parser options from comments and create a corresponding options file.

Test Plan

cargo test

@netlify
Copy link

netlify bot commented Jul 3, 2023

Deploy Preview for docs-rometools ready!

Name Link
🔨 Latest commit c8b5173
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/64a599bedbe9e40007964baf
😎 Deploy Preview https://deploy-preview-4652--docs-rometools.netlify.app/playground
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added A-Parser Area: parser A-Tooling Area: our own build, development, and release tooling labels Jul 3, 2023
@denbezrukov denbezrukov force-pushed the feat/parameter-decorator-test branch from 29e868e to 40a8ac5 Compare July 3, 2023 15:42
@github-actions
Copy link

github-actions bot commented Jul 3, 2023

Parser conformance results on ubuntu-latest

js/262

Test result main count This PR count Difference
Total 48863 48863 0
Passed 47810 47810 0
Failed 1053 1053 0
Panics 0 0 0
Coverage 97.84% 97.84% 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 1764 1764 0
Failed 4448 4448 0
Panics 0 0 0
Coverage 28.40% 28.40% 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 13121 13121 0
Failed 4103 4103 0
Panics 0 0 0
Coverage 76.18% 76.18% 0.00%

@denbezrukov denbezrukov force-pushed the feat/parameter-decorator-test branch 2 times, most recently from 290b126 to 7762ee2 Compare July 3, 2023 19:28
@denbezrukov denbezrukov force-pushed the feat/parameter-decorator-test branch from 7762ee2 to c8b5173 Compare July 5, 2023 16:26
@denbezrukov denbezrukov merged commit 71c70a4 into main Jul 5, 2023
@denbezrukov denbezrukov deleted the feat/parameter-decorator-test branch July 5, 2023 16:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Parser Area: parser A-Tooling Area: our own build, development, and release tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement option-passing mechanism for parser test
2 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载