Skip to content

Conversation

srawlins
Copy link
Member

No description provided.

@google-cla google-cla bot added the cla: yes Google CLA check succeeded. label Jun 12, 2021
@coveralls
Copy link

coveralls commented Jun 14, 2021

Coverage Status

Coverage increased (+0.004%) to 57.626% when pulling 789b3e3 on srawlins:one-sink into bc0d3e2 on dart-lang:master.

@srawlins srawlins requested a review from jcollins-g June 14, 2021 15:34
@srawlins
Copy link
Member Author

This failing test looks very unrelated.

@jcollins-g
Copy link
Contributor

yep failing test looks like more offset retrieval on synthetic accessors. I thought I had fixed that, will look into it today.

Copy link
Contributor

@jcollins-g jcollins-g left a comment

Choose a reason for hiding this comment

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

very nice!

@srawlins srawlins merged commit e5e596c into dart-lang:master Jun 14, 2021
@srawlins srawlins deleted the one-sink branch June 14, 2021 23:04
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