[flake8-simplify] enable fix in preview mode (SIM117)

[VascoSch92]

The PR add the fix safety section for rule SIM117 (#15584 ), and enable a fix in preview mode.

@dylwil3 Is the first time I'm exposed to the preview mode. So please let me know what should I do/fix. I hope I got what you wanted to say.

[github-actions]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

ℹ️ ecosystem check detected linter changes. (+0 -0 violations, +356 -0 fixes in 4 projects; 51 projects unchanged)

apache/airflow (+0 -0 violations, +270 -0 fixes)

ruff check --no-cache --exit-zero --ignore RUF9 --no-fix --output-format concise --preview --select ALL

- airflow-core/src/airflow/cli/commands/rotate_fernet_key_command.py:36:5: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/src/airflow/cli/commands/rotate_fernet_key_command.py:36:5: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/always/test_secrets_local_filesystem.py:380:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/always/test_secrets_local_filesystem.py:380:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/always/test_secrets_local_filesystem.py:392:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/always/test_secrets_local_filesystem.py:392:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/always/test_secrets_local_filesystem.py:406:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/always/test_secrets_local_filesystem.py:406:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/api/common/test_mark_tasks.py:38:5: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/api/common/test_mark_tasks.py:38:5: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/api/common/test_mark_tasks.py:63:5: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/api/common/test_mark_tasks.py:63:5: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/api/common/test_mark_tasks.py:93:5: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/api/common/test_mark_tasks.py:93:5: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_task_instances.py:1669:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_task_instances.py:1669:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/cli/commands/test_dag_command.py:378:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/cli/commands/test_dag_command.py:378:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/core/test_configuration.py:1161:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/core/test_configuration.py:1161:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/core/test_configuration.py:1206:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/core/test_configuration.py:1206:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/core/test_configuration.py:1579:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/core/test_configuration.py:1579:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/core/test_configuration.py:1775:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/core/test_configuration.py:1775:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/core/test_configuration.py:488:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/core/test_configuration.py:488:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/core/test_configuration.py:493:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/core/test_configuration.py:493:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/core/test_configuration.py:695:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/core/test_configuration.py:695:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/core/test_logging_config.py:198:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/core/test_logging_config.py:198:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/core/test_settings.py:223:5: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ airflow-core/tests/unit/core/test_settings.py:223:5: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- airflow-core/tests/unit/core/test_sqlalchemy_config.py:111:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
... 233 additional changes omitted for project

apache/superset (+0 -0 violations, +40 -0 fixes)

ruff check --no-cache --exit-zero --ignore RUF9 --no-fix --output-format concise --preview --select ALL

- superset/db_engine_specs/gsheets.py:376:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ superset/db_engine_specs/gsheets.py:376:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- superset/models/core.py:567:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ superset/models/core.py:567:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- superset/sql_lab.py:706:5: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ superset/sql_lab.py:706:5: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- tests/conftest.py:72:5: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ tests/conftest.py:72:5: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- tests/integration_tests/dashboards/commands_tests.py:776:13: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ tests/integration_tests/dashboards/commands_tests.py:776:13: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
... 30 additional changes omitted for project

bokeh/bokeh (+0 -0 violations, +44 -0 fixes)

ruff check --no-cache --exit-zero --ignore RUF9 --no-fix --output-format concise --preview --select ALL

- src/bokeh/application/handlers/code.py:157:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ src/bokeh/application/handlers/code.py:157:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- tests/unit/bokeh/command/subcommands/test_json__subcommands.py:92:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ tests/unit/bokeh/command/subcommands/test_json__subcommands.py:92:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- tests/unit/bokeh/command/subcommands/test_serve.py:584:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ tests/unit/bokeh/command/subcommands/test_serve.py:584:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- tests/unit/bokeh/command/subcommands/test_serve.py:612:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ tests/unit/bokeh/command/subcommands/test_serve.py:612:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
- tests/unit/bokeh/command/subcommands/test_serve.py:629:9: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ tests/unit/bokeh/command/subcommands/test_serve.py:629:9: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements
... 34 additional changes omitted for project

latchbio/latch (+0 -0 violations, +2 -0 fixes)

ruff check --no-cache --exit-zero --ignore RUF9 --no-fix --output-format concise --preview

- src/latch_cli/services/login.py:38:5: SIM117 Use a single `with` statement with multiple contexts instead of nested `with` statements
+ src/latch_cli/services/login.py:38:5: SIM117 [*] Use a single `with` statement with multiple contexts instead of nested `with` statements

Changes by rule (1 rules affected)

code	total	+ violation	- violation	+ fix	- fix
SIM117	356	0	0	356	0

[dylwil3]

Thank you! This is great - could you add a test as well? You should be able to just copy the same test fixture path here:

ruff/crates/ruff_linter/src/rules/flake8_simplify/mod.rs

Line 61 in a2c87c2

#[test_case(Rule::IfElseBlockInsteadOfIfExp, Path::new("SIM108.py"))]

[VascoSch92]

@dylwil3 thanks for the guidance ;-)

[dylwil3]

Thank you!!