Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

flutter analyze --flutter-repo should detect if user has not run flutter update-packages #37439

Closed
christopherfujino opened this issue Aug 1, 2019 · 3 comments
Closed

flutter analyze --flutter-repo should detect if user has not run flutter update-packages #37439

christopherfujino opened this issue Aug 1, 2019 · 3 comments
Labels
P3 Issues that are less important to the Flutter project tool Affects the "flutter" command-line tool. See also t: labels.

Comments

@christopherfujino
Copy link
Member

Anecdotally, it seems very common to have flutter analyze --flutter-repo to complain about missing import targets, e.g.

Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart'

It would be nice if we could do some kind of check to detect if pub get had ever been run, perhaps check for .packages file?
@christopherfujino christopherfujino added tool Affects the "flutter" command-line tool. See also t: labels. entertaining new contributor project labels Aug 1, 2019
Inconnu08 added a commit to Inconnu08/flutter that referenced this issue Sep 10, 2019
@Inconnu08
Add check for .package file flutter#37439
77ea8f6
@Inconnu08 Inconnu08 mentioned this issue Sep 10, 2019
Closed
Inconnu08 added a commit to Inconnu08/flutter that referenced this issue Sep 10, 2019
@Inconnu08
Add check for .packages file flutter#37439
db5abca
Inconnu08 added a commit to Inconnu08/flutter that referenced this issue Sep 13, 2019
@Inconnu08
Add run update-packages before Analyze-ing flutter#37439
309a71c
Inconnu08 added a commit to Inconnu08/flutter that referenced this issue Sep 14, 2019
@Inconnu08
Add a check to run flutter update-packages only for --flutter-repo fl…
8b794b3 
…utter#37439
Inconnu08 added a commit to Inconnu08/flutter that referenced this issue Sep 30, 2019
@Inconnu08
Add update-packages as part of analyze --flutter-repo flutter#37439
5b5d951
@Inconnu08 Inconnu08 mentioned this issue Sep 30, 2019
Closed
9 tasks
@Inconnu08 Inconnu08 mentioned this issue Nov 27, 2019
Closed
9 tasks
@Hixie Hixie mentioned this issue Dec 2, 2019
Closed
@Hixie Hixie added this to the Future milestone Jan 9, 2020
@kf6gpe kf6gpe added the P6 label May 29, 2020
@kf6gpe kf6gpe removed this from the Future milestone Jun 1, 2020
@Hixie
Copy link
Contributor

Hixie commented Mar 15, 2023

going to close this for now, if we ever want to do it let's just do it but in practice everyone probably just has a script that runs both back to back now anyway.

@Hixie Hixie closed this as completed Mar 15, 2023
@christopherfujino
Copy link
Member Author

Turns out this was not so entertaining after all.

@github-actions
Copy link

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 29, 2023
@flutter-triage-bot flutter-triage-bot bot added P3 Issues that are less important to the Flutter project and removed P6 labels Jun 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
P3 Issues that are less important to the Flutter project tool Affects the "flutter" command-line tool. See also t: labels.
Projects
None yet
Milestone
No milestone
3 participants
@Hixie @kf6gpe @christopherfujino