-
-
Notifications
You must be signed in to change notification settings - Fork 151
refactor: move AspectWatchProtocol to standalone library #2374
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
Conversation
|
176cc1c to
3219992
Compare
95ab2ff to
9716759
Compare
9716759 to
91e56c8
Compare
6d014cc to
a9aad04
Compare
a9aad04 to
dedd304
Compare
dedd304 to
8d7ae75
Compare
|
@codex please review this |
|
Codex Review: Didn't find any major issues. Keep it up! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. |



For some experimental importing+usage of this class in binaries supporting the watch protocol. The goal is minimizing actual changes within rules_js so for now the js_run_devserver.mjs is bundled and is almost identical to before.
Changes are visible to end-users: no
Test plan