Skip to content

Conversation

@pbelevich
Copy link
Contributor

@pbelevich pbelevich commented Oct 25, 2019

Stack from ghstack:

Differential Revision: D18202437

[ghstack-poisoned]
pbelevich added a commit that referenced this pull request Oct 25, 2019
ghstack-source-id: 86f691f
Pull Request resolved: #28654
pbelevich added a commit that referenced this pull request Oct 26, 2019
ghstack-source-id: cde2a06
Pull Request resolved: #28654
Copy link
Contributor

@yf225 yf225 left a comment

Choose a reason for hiding this comment

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

@pbelevich Thanks so much for the awesome work! I left a minor comment.

@yf225 yf225 added the module: cpp Related to C++ API label Oct 29, 2019
@yf225
Copy link
Contributor

yf225 commented Oct 29, 2019

Also it would be awesome to update the parity tracker test/cpp_api_parity/parity-tracker.md for this module as well :D

@facebook-github-bot
Copy link
Contributor

@pbelevich merged this pull request in d6f1e49.

@facebook-github-bot facebook-github-bot deleted the gh/pbelevich/46/head branch November 3, 2019 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: cpp Related to C++ API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants