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.

Sign up for GitHub

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

Jump to bottom

feat(lint): add support for lint plugin selectors #27448

Draft
wants to merge 92 commits into
base: main
Choose a base branch
from
Draft

feat(lint): add support for lint plugin selectors #27448

wants to merge 92 commits into from
+4,984 −85

Conversation

bartlomieju
Copy link
Member

Code factored out from #27203 to make it smaller

bartlomieju and others added 30 commits December 1, 2024 03:21
@bartlomieju
basic plugin loading
79b5c3e
@bartlomieju
basic execution
7e8e4ae
@bartlomieju
some basic structure
1eb4862
@bartlomieju
add visitor
49c2d9e
@bartlomieju
actually pass AST to plugin
99d70c9
@bartlomieju
PluginRunner and PluginRunnerProxy
1fdcbd9
@bartlomieju
req -> resp
bc00517
@bartlomieju
first diagnostic from a plugin!
7f45875
@bartlomieju
load plugins from a flag
c9fa293
@bartlomieju
declarative
5639379
@bartlomieju
cleanup
0b36971
@bartlomieju
basic range
f9291e4
@bartlomieju
measure how long i takes to run
dea3219
@bartlomieju
don't dump ast
439d43e
@bartlomieju
Merge branch 'main' into lint_plugins
c3208df
@bartlomieju
add basic traversal
5a6c128
@bartlomieju
correctly report
fb47288
@bartlomieju
remove some prints
a15c116
@marvinhagemeister
drop!: vue demo rule
008e249
@marvinhagemeister
feat: add source() method
f3745f9
@marvinhagemeister
chore: add more lint rules
4676643
@bartlomieju
build on stable rust
1217314
@bartlomieju
remove unneeded files
a1dfea6
@bartlomieju
remove debug log, lint
37bafdc
@marvinhagemeister
fix: show correct specifier in error output
77526ca
@marvinhagemeister
drop!: temporary disable lint cache
5a950e5
@bartlomieju
wip
42d9936
@bartlomieju
Merge branch 'main' into lint_plugins
c2ac424
@bartlomieju
load plugins from a config file
106b1f1
@bartlomieju
almost working with regular cli worker
3e1e46d
marvinhagemeister and others added 30 commits December 12, 2024 10:22
@marvinhagemeister
more_nodes
dd75c31
@marvinhagemeister
refactors
d9e506e
@marvinhagemeister
fix: clippy
dda408b
@marvinhagemeister
refactor ast
2902eba
@marvinhagemeister
basic ts nodes
7c7a888
@marvinhagemeister
more nodes
8f10431
@marvinhagemeister
fix traversal
76c1275
@marvinhagemeister
Refactor binary ast
c83f722
@marvinhagemeister
refactor node handling
f50bf9b
@marvinhagemeister
improve format
65fcda4
@marvinhagemeister
make js plugins independent of ast strucure
ea11f95
@marvinhagemeister
feat(lint): support visitor selector syntax
3f9ee0d
@marvinhagemeister
remove dead code
f167277
@marvinhagemeister
WIP
86fc6e6
@marvinhagemeister
WIP
2f18594
@marvinhagemeister
fix selector parser
e542ba8
@marvinhagemeister
fix nth child
132db56
@marvinhagemeister
integrate selctors part 1
d9fd959
@marvinhagemeister
get selector integration working
6e70ba2
@marvinhagemeister
fix: descendant selector
dde9f52
@bartlomieju
Merge branch 'main' into lint_plugins
53a680b
@marvinhagemeister
fix: node name
8d9f9ce
@marvinhagemeister
fix: sibling selectors
321a1fe
@marvinhagemeister
fix more types
e296022
@marvinhagemeister
improve conversion safety
5ea018a
@marvinhagemeister
clippy stuff
13cc74d
@bartlomieju
Merge branch 'main' into lint_plugins
d3e41a7
@bartlomieju
wip after merge
fcfd4c0
@bartlomieju
Try to load files differently
67e2d06
@bartlomieju
runs tests, but they fail
6e28ae5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bartlomieju @marvinhagemeister