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: remove try_cast_node #3521

Merged
merged 1 commit into from
Jul 25, 2024
Merged

Conversation

Conaclos
Copy link
Member

Summary

Follow-up of #3520
The PR removes try_cast_node (only used once) and replace some try_cast with cast when it makes sense.

Test Plan

CI must pass.

@github-actions github-actions bot added A-Core Area: core A-Linter Area: linter L-JavaScript Language: JavaScript and super languages L-JSON Language: JSON and super languages labels Jul 25, 2024
Copy link
Contributor

Parser conformance results on

js/262

Test result main count This PR count Difference
Total 48369 48369 0
Passed 47172 47172 0
Failed 1197 1197 0
Panics 0 0 0
Coverage 97.53% 97.53% 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 6544 6544 0
Passed 2197 2197 0
Failed 4347 4347 0
Panics 0 0 0
Coverage 33.57% 33.57% 0.00%

ts/babel

Test result main count This PR count Difference
Total 670 670 0
Passed 598 598 0
Failed 72 72 0
Panics 0 0 0
Coverage 89.25% 89.25% 0.00%

ts/microsoft

Test result main count This PR count Difference
Total 18313 18313 0
Passed 14008 14008 0
Failed 4305 4305 0
Panics 0 0 0
Coverage 76.49% 76.49% 0.00%

@Conaclos Conaclos merged commit 112e6d7 into main Jul 25, 2024
15 checks passed
@Conaclos Conaclos deleted the conaclos/remove-try-cast-node branch July 25, 2024 15:52
Copy link

codspeed-hq bot commented Jul 25, 2024

CodSpeed Performance Report

Merging #3521 will not alter performance

Comparing conaclos/remove-try-cast-node (0569e7a) with main (ff21256)

Summary

✅ 104 untouched benchmarks

@Conaclos Conaclos added the A-Changelog Area: changelog label Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Changelog Area: changelog A-Core Area: core A-Linter Area: linter L-JavaScript Language: JavaScript and super languages L-JSON Language: JSON and super languages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant