refactor(linter/plugins): add parse function#16240
Conversation
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
CodSpeed Performance ReportMerging #16240 will not alter performanceComparing Summary
Footnotes
|
6f782ba to
b184e4f
Compare
b184e4f to
1d22ec5
Compare
1d22ec5 to
43d7fef
Compare
2d68a5a to
7150209
Compare
43d7fef to
96f4c5b
Compare
96f4c5b to
4318041
Compare
4318041 to
eb44265
Compare
eb44265 to
fa5947d
Compare
65cd570 to
c56d44d
Compare
fa5947d to
26e3bdd
Compare
c56d44d to
72b48e7
Compare
Merge activity
|
Add function to parse source text into raw transfer buffer to `apps/oxlint`. It's required for rule tester (#16206).
72b48e7 to
1af1091
Compare
26e3bdd to
db2afba
Compare
There was a problem hiding this comment.
is this code the same as oxc parser napi package? should the code between the two be shared?
There was a problem hiding this comment.
It's mostly the same, but has some differences to integrate with the slight difference between oxc-parser and oxlint (Oxlint doesn't deal with the module record).
Is there a way to have code shared between the 2 packages, without a dependency? Maybe if we convert napi/parser to TS, and build it with TSDown. (speaking of converting, yes, I'm a convert to TS)
There was a problem hiding this comment.
Is there a way to have code shared between the 2 packages, without a dependency?
best way it just to have it as a dep, we can make sure that tsdown bundles it so there'll be no runtime overhead
Add function to parse source text into raw transfer buffer to `apps/oxlint`. It's required for rule tester (oxc-project#16206).

Add function to parse source text into raw transfer buffer to
apps/oxlint. It's required for rule tester (#16206).