Skip to content

operator== becomes == #1287

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

Closed
wants to merge 1 commit into from
Closed

operator== becomes == #1287

wants to merge 1 commit into from

Conversation

jcollins-g
Copy link
Contributor

Possibly the result of an analyzer change beneath us, it looks like how the == operator method is represented has changed. @keertip @devoncarew

@googlebot googlebot added the cla: yes Google CLA check succeeded. label Dec 9, 2016
@jcollins-g jcollins-g changed the title operator== becomes operator operator== becomes == Dec 9, 2016
@devoncarew
Copy link
Member

Interesting - we're using the same analyzer, but generating against two different SDKs (the current stable and the dev channel SDK). We're working against dev but not against stable.

@devoncarew
Copy link
Member

I think you can make travis happy by only running the tests against the dev channel. Commenting out this line: https://github.com/dart-lang/dartdoc/blob/master/.travis.yml#L5 - should do it. Let's file an issue to re-enable once dev goes stable.

lgtm once travis goes green; thanks!

@jcollins-g
Copy link
Contributor Author

Looks like you guys already did this part.

@jcollins-g jcollins-g closed this Dec 15, 2016
@jcollins-g jcollins-g deleted the operator_change branch December 15, 2016 18:38
@devoncarew
Copy link
Member

Yup, but thanks anyway! (we had an important change to get in and wanted the bots green for it)

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.

3 participants