Skip to content

Update analyzer, fix checked mode runs for flutter, and unpin bots. #1606

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

Merged
merged 7 commits into from
Feb 22, 2018

Conversation

jcollins-g
Copy link
Contributor

@jcollins-g jcollins-g commented Feb 14, 2018

Fixes #1602.

Checked mode failures were due to a bug in the markdown processor. Add a test for that change, update analyzer (required for checked mode to work), and unpin travis/appveyor so we again use the latest SDK builds.

@googlebot googlebot added the cla: yes Google CLA check succeeded. label Feb 14, 2018
@jcollins-g jcollins-g changed the title Update analyzer and fix checked mode runs for flutter and unpin Travis. Update analyzer, fix checked mode runs for flutter, and unpin Travis. Feb 14, 2018
@jcollins-g jcollins-g changed the title Update analyzer, fix checked mode runs for flutter, and unpin Travis. Update analyzer, fix checked mode runs for flutter, and unpin bots. Feb 14, 2018
Copy link
Member

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Drive-by-comment

@@ -1,7 +1,7 @@
language: dart
sudo: false
dart:
- "dev/release/2.0.0-dev.22.0"
- "dev/raw/latest"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just dev here!

Copy link
Contributor Author

@jcollins-g jcollins-g Feb 15, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm actually looking for raw here. Not all versions that we actually wind up using in internal rolls, etc, get published to release right away, and I'd like to have the bots go red on a problem ahead of that.

appveyor.yml Outdated
@@ -3,7 +3,7 @@
# BSD-style license that can be found in the LICENSE file.

install:
- ps: wget https://gsdview.appspot.com/dart-archive/channels/dev/raw/2.0.0-dev.22.0/sdk/dartsdk-windows-x64-release.zip -OutFile dart-sdk.zip
- ps: wget https://gsdview.appspot.com/dart-archive/channels/dev/raw/latest/sdk/dartsdk-windows-x64-release.zip -OutFile dart-sdk.zip
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be https://storage.googleapis.com/dart-archive/channels/dev/release/latest/sdk/dartsdk-windows-x64-release.zip

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the "official" external URI

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see above

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At least change the domain. gsdview.appspot.com is a weird internal tool that happens to work.

No promises that it will stick around. It adds a layer on to downloading the SDK, etc.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, yeah, good idea. No idea why it was this way in the first place. Done.

@@ -427,7 +427,7 @@ Future<List<Map>> _buildFlutterDocs(
workingDirectory: await futureCwd);
return await launcher.runStreamed(
flutterCacheDart,
[path.join('dev', 'tools', 'dartdoc.dart'), '--json'],
[path.join('dev', 'tools', 'dartdoc.dart'), '-c', '--json'],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 awesome

@jcollins-g jcollins-g merged commit 3cb8548 into master Feb 22, 2018
@kevmoo kevmoo deleted the fix-checked-mode branch February 26, 2018 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Google CLA check succeeded.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants