Mentions légales du service

Skip to content
Snippets Groups Projects

Make rustfmt config path an `Option`, remove argument if not provided

Merged GONZALEZ Amélie requested to merge ngonzale/coccinelleforrust:rustfmt-config-file into main

If no path to a rustfmt configuration file was provided, this program would default to rustfmt.toml, and force rustfmt to try and find it recursively in all folders, even if it did not end up finding it.

This patch makes the rustfmt configuration file path an Option, meaning that no default value is provided. If no file path is given, the --config-path option is never provided to rustfmt during its invocation.

(I also moved three lines of code that should be triggered only if rustfmt runs, they were not hidden by the suppressor flag in !6 (merged) )

Merge request reports

Approved by

Merged by Tathagata RoyTathagata Roy 1 year ago (Aug 15, 2023 11:33am UTC)

Merge details

  • Changes merged into main with e9409d0a.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading