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.

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

fix analysis warnings in imitation_game #234

Merged
merged 1 commit into from
Oct 24, 2020

Conversation

yjbanov
Copy link
Contributor

@yjbanov yjbanov commented Oct 23, 2020

This should fix the master branch build.

/cc @creativecreatorormaybenot

@yjbanov yjbanov requested a review from gaaclarke October 23, 2020 22:57
@gaaclarke
Copy link
Member

Thanks, weird that it didn't need this change:

--- a/packages/imitation_game/bin/imitation_game.dart
+++ b/packages/imitation_game/bin/imitation_game.dart
@@ -5,8 +5,6 @@ import 'package:mustache/mustache.dart';
 import 'package:imitation_game/README_template.dart';
 import 'package:args/args.dart';
 
-// ignore_for_file: avoid_as
-
 const int _port = 4040;
 
 Future<String> _findIpAddress() async {

@creativecreatorormaybenot
Copy link
Contributor

I suppose it is due to an external analyzer change.

@gaaclarke
Copy link
Member

@creativecreatorormaybenot yep, it would be nice if those changes were incorporated as commits too so we could catch it earlier. Here's another reason why having a mega-repository would be great (https://cacm.acm.org/magazines/2016/7/204032-why-google-stores-billions-of-lines-of-code-in-a-single-repository/fulltext). I just had to fix the tree the other day after someone else's commit, now Yegor fixed it for me, karma.

@yjbanov
Copy link
Contributor Author

yjbanov commented Oct 24, 2020

I know. Monorepo FTW! I'm not sure what's up with ignore_for_file: avoid_as but if I remove it the analyzer complains 🤷‍♂️

@yjbanov
Copy link
Contributor Author

yjbanov commented Oct 24, 2020

Landing on red to (hopefully) fix the build.

@yjbanov
Copy link
Contributor Author

yjbanov commented Oct 24, 2020

Filed flutter/flutter#68934

stuartmorgan pushed a commit to stuartmorgan/packages that referenced this pull request Apr 30, 2021
stuartmorgan pushed a commit that referenced this pull request Oct 31, 2024
* Use package http

* delete test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants