Tools: πŸ“Ž Reduce command suggestion console noise

Created on 11 Feb 2021  Β·  2Comments  Β·  Source: rome/tools

 internal/path/index.ts lint/pendingFixes  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  βœ– Pending formatting and safe fixes

    1087 1087 β”‚   }
    1088 1088 β”‚
    1089      β”‚ - exportΒ·constΒ·UNKNOWN_PATHΒ·=Β·createUIDPath("unknown");
         1089 β”‚ + exportΒ·constΒ·UNKNOWN_PATHΒ·=Β·createUIDPath("unknown");
         1090 β”‚ +

  β„Ή Only difference is leading and trailing whitespace

  β„Ή To apply fixes and formatting run
  $ rome check internal/path/index.ts --apply


━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

β„Ή Fixes available. To apply safe fixes and formatting run
$ rome check --apply
β„Ή To choose fix suggestions run
$ rome check --review

βœ– Found 6 problems

We suggest too many commands. Not sure what the right balance is.

task

Most helpful comment

I'd remove the per diagnostic command suggestion and just keep the 2 at the end (and maybe change the wording/add spacing).

 internal/path/index.ts lint/pendingFixes  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  βœ– Pending formatting and safe fixes

    1087 1087 β”‚   }
    1088 1088 β”‚
    1089      β”‚ - exportΒ·constΒ·UNKNOWN_PATHΒ·=Β·createUIDPath("unknown");
         1089 β”‚ + exportΒ·constΒ·UNKNOWN_PATHΒ·=Β·createUIDPath("unknown");
         1090 β”‚ +

  β„Ή Only difference is leading and trailing whitespace


━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

β„Ή Fixes available. To apply safe fixes and formatting run
$ rome check --apply

β„Ή To interactively choose fix suggestions run
$ rome check --review


βœ– Found 6 problems

All 2 comments

It would be great to receive some feedback from the adopter!

I'd remove the per diagnostic command suggestion and just keep the 2 at the end (and maybe change the wording/add spacing).

 internal/path/index.ts lint/pendingFixes  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  βœ– Pending formatting and safe fixes

    1087 1087 β”‚   }
    1088 1088 β”‚
    1089      β”‚ - exportΒ·constΒ·UNKNOWN_PATHΒ·=Β·createUIDPath("unknown");
         1089 β”‚ + exportΒ·constΒ·UNKNOWN_PATHΒ·=Β·createUIDPath("unknown");
         1090 β”‚ +

  β„Ή Only difference is leading and trailing whitespace


━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

β„Ή Fixes available. To apply safe fixes and formatting run
$ rome check --apply

β„Ή To interactively choose fix suggestions run
$ rome check --review


βœ– Found 6 problems
Was this page helpful?
0 / 5 - 0 ratings

Related issues

sebmck picture sebmck  Β·  3Comments

cavoirom picture cavoirom  Β·  4Comments

sebmck picture sebmck  Β·  5Comments

yamadayuki picture yamadayuki  Β·  3Comments

linvain picture linvain  Β·  4Comments