Skip to content

Conversation

@heitorschueroff
Copy link
Contributor

@heitorschueroff heitorschueroff commented Aug 5, 2020

Stack from ghstack:

Differential Revision: D23030302

heitorschueroff added a commit that referenced this pull request Aug 5, 2020
ghstack-source-id: d50438b
Pull Request resolved: #42613
@glaringlee glaringlee requested review from glaringlee and removed request for ebetica and goldsborough August 5, 2020 18:55
@dr-ci
Copy link

dr-ci bot commented Aug 5, 2020

💊 CI failures summary and remediations

As of commit 4cc98f3 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 54 times.

Copy link
Contributor

@glaringlee glaringlee left a comment

Choose a reason for hiding this comment

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

Commented, please fix the clang-tidy as well.

heitorschueroff added a commit that referenced this pull request Aug 6, 2020
ghstack-source-id: fa30b0e
Pull Request resolved: #42613
Copy link
Contributor

@glaringlee glaringlee left a comment

Choose a reason for hiding this comment

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

@heitorschueroff please fix clang-tidy lint error.

heitorschueroff added a commit that referenced this pull request Aug 7, 2020
ghstack-source-id: 838d5a6
Pull Request resolved: #42613
Copy link
Contributor

@glaringlee glaringlee left a comment

Choose a reason for hiding this comment

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

LGTM now.

heitorschueroff added a commit that referenced this pull request Aug 11, 2020
ghstack-source-id: 47407bb
Pull Request resolved: #42613
heitorschueroff added a commit that referenced this pull request Aug 11, 2020
ghstack-source-id: ad26fd8
Pull Request resolved: #42613
heitorschueroff added a commit that referenced this pull request Aug 12, 2020
ghstack-source-id: 3656609
Pull Request resolved: #42613
@facebook-github-bot
Copy link
Contributor

@heitorschueroff merged this pull request in 3d8c144.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants