-
Notifications
You must be signed in to change notification settings - Fork 14
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
Source location #70
Source location #70
Conversation
Codecov Report
@@ Coverage Diff @@
## master #70 +/- ##
===========================================
- Coverage 86.19% 74.60% -11.59%
===========================================
Files 15 8 -7
Lines 8380 4907 -3473
===========================================
- Hits 7223 3661 -3562
- Misses 1157 1246 +89
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
This was expected to impact performance at some level, below is a preliminary benchmark to get a good understanding of how performance will differ from the currently released version.
While the time it takes to run these benchmarks seems to be increasing by a factor of ~2, which warrants investigation, the performance hasn't dropped to be outlandishly slow. |
97baac4
to
3be36af
Compare
3be36af
to
bc435a7
Compare
5aecef4
to
1ebb72f
Compare
This updates the parser to use an AST that has attached source locations for each node to allow for generating things like source maps