Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update decaffeinate-parser example to latest to fix build #691

Merged
merged 1 commit into from
Apr 3, 2022

Conversation

alangpierce
Copy link
Owner

This pulls in a fix for a dependency resolution failure related to coffee-script-redux. The project has switched to jest in the
meantime, so this updates the patch to use the jest integration.

@codecov
Copy link

codecov bot commented Apr 3, 2022

Codecov Report

Merging #691 (e8ecff6) into main (c350695) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #691   +/-   ##
=======================================
  Coverage   82.59%   82.59%           
=======================================
  Files          56       56           
  Lines        5694     5694           
  Branches     1343     1343           
=======================================
  Hits         4703     4703           
  Misses        708      708           
  Partials      283      283           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@github-actions
Copy link

github-actions bot commented Apr 3, 2022

Benchmark results

Before this PR: 283.1 thousand lines per second
After this PR: 284 thousand lines per second

Measured change: 0.33% faster (1.36% slower to 0.65% faster)
Summary: Likely no significant difference

@alangpierce alangpierce merged commit e084f7a into main Apr 3, 2022
@alangpierce alangpierce deleted the update-decaffeinate-parser-to-latest branch April 3, 2022 21:16
1Lighty pushed a commit to Astra-mod/sucrase that referenced this pull request Aug 14, 2022
…e#691)

This pulls in a fix for a dependency resolution failure related to coffee-script-redux. The project has switched to jest in the
meantime, so this updates the patch to use the jest integration.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant