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

fix: revert using acorn tokenizer #65

Closed
wants to merge 3 commits into from
Closed

Conversation

danielroe
Copy link
Member

This reverts commit 7039f54.

resolves #64.

If this is an acceptable hotfix I'll reopen the issue the previous commit solved and we can look to reenable the feature, perhaps in a different way.This reverts commit 7039f54.

@danielroe danielroe added the bug Something isn't working label Aug 3, 2022
@danielroe danielroe requested a review from pi0 August 3, 2022 15:30
@danielroe danielroe self-assigned this Aug 3, 2022
@codecov
Copy link

codecov bot commented Aug 3, 2022

Codecov Report

Merging #65 (be0a93a) into main (83e30de) will decrease coverage by 0.27%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #65      +/-   ##
==========================================
- Coverage   58.05%   57.78%   -0.28%     
==========================================
  Files          13       13              
  Lines        2172     2127      -45     
  Branches      218      210       -8     
==========================================
- Hits         1261     1229      -32     
+ Misses        701      696       -5     
+ Partials      210      202       -8     
Impacted Files Coverage Δ
src/analyze.ts 81.25% <ø> (+2.41%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@pi0
Copy link
Member

pi0 commented Aug 3, 2022

Fixed by 4ed5c61 using fallback

@pi0 pi0 closed this Aug 3, 2022
@pi0 pi0 deleted the fix/revert-tokenizer branch August 3, 2022 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

parsing issues with latest release
2 participants