-
Notifications
You must be signed in to change notification settings - Fork 9.8k
Conversation
Analysis issues around constructor order are also showing up in head, and look related to a Dart roll, not this PR. |
LGTM, but let's get the CI job working in head before we merge this |
cc @cbracken is someone looking at the CI issue? |
I'm fixing the CI issues. |
84e34e5
to
ffbf999
Compare
As the build failures are now the same as in master (and as this PR only touch READMEs and docs) I'm tempted to merge this PR. WDYT? |
@cbracken can we land this? |
Merging as CI is passing now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks |
@a14n looks like the CI job failed after the merge; does this look related to your changes? |
I don't think it's related to my PR (only README and comments had been changed). |
This reverts commit 5d07373.
No description provided.