refactor(linter/plugins): remove experimental warning#18771
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. |
There was a problem hiding this comment.
Pull request overview
This PR removes the experimental warning message that was displayed to stderr when JS plugins were loaded in oxlint. The warning informed users that JS plugins are experimental and not subject to semver. As the team prepares for the alpha release, this warning is being removed to reduce noise while the plugins remain technically experimental.
Changes:
- Removed the warning code that printed to stderr on first plugin load
- Updated 66 snapshot test files to reflect the absence of the warning in stderr output
Reviewed changes
Copilot reviewed 64 out of 64 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| crates/oxc_linter/src/config/config_builder.rs | Removed the conditional check and eprintln statement (9 lines) that displayed the experimental warning, along with the #[expect(clippy::print_stderr)] attribute |
| apps/oxlint/test/fixtures/*/output.snap.md (66 files) | Updated snapshot tests to remove the expected warning text from stderr sections |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Merging this PR will not alter performance
Comparing Footnotes
|
6e989e2 to
b72e579
Compare
e13fa60 to
8374b33
Compare
8374b33 to
922d883
Compare
b72e579 to
5c042fa
Compare
922d883 to
85c0c93
Compare
5c042fa to
0d2999d
Compare
0d2999d to
ccdaaaa
Compare
85c0c93 to
1e160fc
Compare
1e160fc to
5a93dfc
Compare
ccdaaaa to
53b5f18
Compare
|
@camc314 can probably split this out of the stack and merge it alone? |
5a93dfc to
d981978
Compare
53b5f18 to
a549b21
Compare
Merge activity
|
As we prepare for alpha release of JS plugins, it feels like we can remove the warning logged to stderr. ``` WARNING: JS plugins are experimental and not subject to semver. Breaking changes are possible while JS plugins support is under development. ``` They do still remain experimental, but I don't think we need to shout that quite as loudly now.
a549b21 to
6ad4480
Compare

As we prepare for alpha release of JS plugins, it feels like we can remove the warning logged to stderr.
They do still remain experimental, but I don't think we need to shout that quite as loudly now.