Skip to content
This repository has been archived by the owner on Dec 14, 2018. It is now read-only.

React to breaking change latest API Check detects #6322

Closed
wants to merge 1 commit into from
Closed

React to breaking change latest API Check detects #6322

wants to merge 1 commit into from

Conversation

dougbu
Copy link
Member

@dougbu dougbu commented May 25, 2017

@dougbu
Copy link
Member Author

dougbu commented May 25, 2017

@SteveSandersonMS this relates to one of your changes and I already know it was intentional. But, please double-check the change here.

@dougbu dougbu requested a review from SteveSandersonMS May 25, 2017 05:12
SteveSandersonMS
SteveSandersonMS approved these changes May 25, 2017
View reviewed changes
Copy link
Member

@SteveSandersonMS SteveSandersonMS left a comment

Choose a reason for hiding this comment

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

Looks good to me

@dougbu dougbu force-pushed the dougbu/api.check.reaction branch 2 times, most recently from 1f88139 to 4e3c469 Compare May 25, 2017 21:35
@dougbu dougbu force-pushed the dougbu/api.check.reaction branch from 4e3c469 to 1a294b8 Compare May 26, 2017 04:27
@dougbu
Copy link
Member Author

dougbu commented May 26, 2017

9f4ff22

@dougbu dougbu closed this May 26, 2017
@dougbu dougbu deleted the dougbu/api.check.reaction branch May 26, 2017 16:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@SteveSandersonMS SteveSandersonMS SteveSandersonMS approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dougbu @SteveSandersonMS @dnfclas