fix(core): file size diagnostic severity#9980
Conversation
🦋 Changeset detectedLatest commit: 2647031 The changes in this PR will be included in the next version bump. This PR includes changesets to release 13 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
WalkthroughThis change addresses issue Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.changeset/every-poets-throw.md:
- Line 5: Update the changeset sentence to correct the grammar: replace "file
exceed the `files.maxSize` limit" with "file exceeds the `files.maxSize` limit"
so the user-facing line reads "Biome now emits a `warning` diagnostic when a
file exceeds the `files.maxSize` limit."; ensure the rest of the changeset text
and tense follow the project's guideline for concise, clear descriptions.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: c56bb05f-8536-4dba-8aa7-52628dc20ee9
⛔ Files ignored due to path filters (15)
crates/biome_cli/tests/snapshots/main_cases_overrides_max_file_size/overrides_files_max_size_too_large_limit_check.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_cases_overrides_max_file_size/overrides_files_max_size_too_large_limit_format.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_check/file_too_large.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_check/file_too_large_cli_limit.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_check/file_too_large_config_limit.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_check/file_too_large_error_on_warnings.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_ci/file_too_large.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_ci/file_too_large_cli_limit.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_ci/file_too_large_config_limit.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_format/file_too_large.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_format/file_too_large_cli_limit.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_format/file_too_large_config_limit.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_lint/file_too_large.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_lint/file_too_large_cli_limit.snapis excluded by!**/*.snapand included by**crates/biome_cli/tests/snapshots/main_commands_lint/file_too_large_config_limit.snapis excluded by!**/*.snapand included by**
📒 Files selected for processing (3)
.changeset/every-poets-throw.mdcrates/biome_cli/tests/commands/check.rscrates/biome_service/src/diagnostics.rs
Summary
Closes #9941
Test Plan
Added a new test
Docs