Remove experimental formatter warning (#8148)

Co-authored-by: Micha Reiser <micha@reiser.io>
This commit is contained in:
Charlie Marsh 2023-10-23 18:26:29 -07:00 committed by GitHub
parent 2f32a57cf4
commit 7f4ea6690d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 1 additions and 16 deletions

View file

@ -437,7 +437,7 @@ pub(crate) enum FormatResult {
/// The file was unchanged, as the formatted contents matched the existing contents. /// The file was unchanged, as the formatted contents matched the existing contents.
Unchanged, Unchanged,
/// Skipped formatting because its an unformatted file format /// Skipped formatting because its an unsupported file format
Skipped, Skipped,
} }

View file

@ -172,8 +172,6 @@ pub fn run(
} }
fn format(args: FormatCommand, log_level: LogLevel) -> Result<ExitStatus> { fn format(args: FormatCommand, log_level: LogLevel) -> Result<ExitStatus> {
warn_user_once!("`ruff format` is not yet stable, and subject to change in future versions.");
let (cli, overrides) = args.partition(); let (cli, overrides) = args.partition();
if is_stdin(&cli.files, cli.stdin_filename.as_deref()) { if is_stdin(&cli.files, cli.stdin_filename.as_deref()) {

View file

@ -40,7 +40,6 @@ if condition:
print('Hy "Micha"') # Should not change quotes print('Hy "Micha"') # Should not change quotes
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
"###); "###);
} }
@ -89,7 +88,6 @@ if condition:
print('Should change quotes') print('Should change quotes')
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
"###); "###);
Ok(()) Ok(())
} }
@ -117,7 +115,6 @@ fn mixed_line_endings() -> Result<()> {
----- stdout ----- ----- stdout -----
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
2 files left unchanged 2 files left unchanged
"###); "###);
Ok(()) Ok(())
@ -187,7 +184,6 @@ OTHER = "OTHER"
2 files would be reformatted 2 files would be reformatted
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
"###); "###);
Ok(()) Ok(())
} }
@ -221,7 +217,6 @@ if __name__ == '__main__':
----- stdout ----- ----- stdout -----
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
"###); "###);
Ok(()) Ok(())
} }
@ -278,7 +273,6 @@ if condition:
print('Should change quotes') print('Should change quotes')
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
warning: The following rules may cause conflicts when used with the formatter: 'Q000'. To avoid unexpected behavior, we recommend disabling these rules, either by removing them from the `select` or `extend-select` configuration, or adding then to the `ignore` configuration. warning: The following rules may cause conflicts when used with the formatter: 'Q000'. To avoid unexpected behavior, we recommend disabling these rules, either by removing them from the `select` or `extend-select` configuration, or adding then to the `ignore` configuration.
"###); "###);
Ok(()) Ok(())
@ -360,7 +354,6 @@ def say_hy(name: str):
1 file reformatted 1 file reformatted
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
warning: The following rules may cause conflicts when used with the formatter: 'COM812', 'COM819', 'D206', 'E501', 'ISC001', 'Q000', 'Q001', 'Q002', 'Q003', 'W191'. To avoid unexpected behavior, we recommend disabling these rules, either by removing them from the `select` or `extend-select` configuration, or adding then to the `ignore` configuration. warning: The following rules may cause conflicts when used with the formatter: 'COM812', 'COM819', 'D206', 'E501', 'ISC001', 'Q000', 'Q001', 'Q002', 'Q003', 'W191'. To avoid unexpected behavior, we recommend disabling these rules, either by removing them from the `select` or `extend-select` configuration, or adding then to the `ignore` configuration.
warning: The following isort options may cause conflicts when used with the formatter: 'isort.force-single-line', 'isort.force-wrap-aliases', 'isort.lines-after-imports', 'isort.lines_between_types'. To avoid unexpected behavior, we recommend disabling these options by removing them from the configuration. warning: The following isort options may cause conflicts when used with the formatter: 'isort.force-single-line', 'isort.force-wrap-aliases', 'isort.lines-after-imports', 'isort.lines_between_types'. To avoid unexpected behavior, we recommend disabling these options by removing them from the configuration.
"###); "###);
@ -400,7 +393,6 @@ def say_hy(name: str):
print(f"Hy {name}") print(f"Hy {name}")
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
warning: The following rules may cause conflicts when used with the formatter: 'COM812', 'COM819', 'D206', 'E501', 'ISC001', 'Q000', 'Q001', 'Q002', 'Q003', 'W191'. To avoid unexpected behavior, we recommend disabling these rules, either by removing them from the `select` or `extend-select` configuration, or adding then to the `ignore` configuration. warning: The following rules may cause conflicts when used with the formatter: 'COM812', 'COM819', 'D206', 'E501', 'ISC001', 'Q000', 'Q001', 'Q002', 'Q003', 'W191'. To avoid unexpected behavior, we recommend disabling these rules, either by removing them from the `select` or `extend-select` configuration, or adding then to the `ignore` configuration.
warning: The following isort options may cause conflicts when used with the formatter: 'isort.force-single-line', 'isort.force-wrap-aliases', 'isort.lines-after-imports', 'isort.lines_between_types'. To avoid unexpected behavior, we recommend disabling these options by removing them from the configuration. warning: The following isort options may cause conflicts when used with the formatter: 'isort.force-single-line', 'isort.force-wrap-aliases', 'isort.lines-after-imports', 'isort.lines_between_types'. To avoid unexpected behavior, we recommend disabling these options by removing them from the configuration.
"###); "###);
@ -443,7 +435,6 @@ fn test_diff() {
z = 3 z = 3
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
2 files would be reformatted, 1 file left unchanged 2 files would be reformatted, 1 file left unchanged
"###); "###);
}); });
@ -473,7 +464,6 @@ fn test_diff_no_change() {
z = 3 z = 3
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
1 file would be reformatted 1 file would be reformatted
"### "###
); );
@ -507,7 +497,6 @@ fn test_diff_stdin_unformatted() {
z = 3 z = 3
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
"###); "###);
} }
@ -524,6 +513,5 @@ fn test_diff_stdin_formatted() {
----- stdout ----- ----- stdout -----
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
"###); "###);
} }

View file

@ -485,7 +485,6 @@ fn stdin_format_jupyter() {
} }
----- stderr ----- ----- stderr -----
warning: `ruff format` is not yet stable, and subject to change in future versions.
"###); "###);
} }