Cleanup flutter analyze command defaults. #4387

Activity

1. pq

   added

   affects: dev experience

   c: new featureNothing broken; request for a new capabilityNothing broken; request for a new capability

   toolAffects the "flutter" command-line tool. See also t: labels.Affects the "flutter" command-line tool. See also t: labels.

   on Jun 6, 2016
2. pq

   added this to the Flutter 1.0This milestone has been deleted milestone on Jun 6, 2016
3. devoncarew commented on Jun 6, 2016

   

   devoncarew

   on Jun 6, 2016

   Contributor

   More actions

   Usage: flutter analyze [arguments]
   -h, --help                      Print this usage information.
       --[no-]flutter-repo         Include all the examples and tests from the Flutter repository.
       --[no-]current-directory    Include all the Dart files in the current directory, if any.
                                   (defaults to on)
   
       --[no-]current-package      Include the lib/main.dart file from the current directory, if any.
                                   (defaults to on)
   
       --[no-]dartdocs             List every public member that is lacking documentation (only examines files in the Flutter repository).
       --[no-]preamble             Display the number of files that will be analyzed.
                                   (defaults to on)
   
       --[no-]congratulate         Show output even when there are no errors, warnings, hints, or lints.
                                   (defaults to on)
   
       --watch                     Run analysis continuously, watching the filesystem for changes.
       --[no-]pub                  Whether to run "pub get" before executing this command.
                                   (defaults to on)
   

   I wonder if we could eliminate some of these options entirely, and simplify the matrix of possibilities a bit?

   👍React with 👍1Reacted by Phil Quitslund
4. pq commented on Jun 6, 2016

   

   pq

   on Jun 6, 2016

   ContributorAuthor

   More actions

   --[no-]current-directory    Include all the Dart files in the current directory, if any.
                               (defaults to on)
   

   Maybe this can be removed given that we should get the equivalent like so:

   flutter analyze .

   (Bonus points for being shorter than flutter analyze --current-directory 😄 )
5. Hixie

   removed

   old label "dev experience" (tell Hixie today if you don't want this label deleted)

   on Jan 22, 2017
6. Hixie

   modified the milestones: 3: Make conferences happyThis milestone has been deleted, 5: Make Hixie proudThis milestone has been deleted on Jan 30, 2017
7. Hixie commented on Jan 30, 2017

   

   Hixie

   on Jan 30, 2017

   Contributor

   More actions

   Closing in favour of #7274
8. Hixie

   closed this as completedon Jan 30, 2017
9. cdotstout

   added a commit that references this issue on Apr 3, 2018

   Revert "Move texture registry ownership to platform view (flutter#4348)…

   ...

   1e6ebb3
10. github-actions commented on Sep 5, 2021

    

    github-actionsbot

    on Sep 5, 2021

    More actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.
11. github-actions

    locked as resolved and limited conversation to collaborators on Sep 5, 2021