-
Notifications
You must be signed in to change notification settings - Fork 555
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
Address nits from previous PR stack + bump version #518
Conversation
[ghstack-poisoned]
ghstack-source-id: 3f1beaf038a5375fa625eec3dc45f079e35afe48 Pull Request resolved: #518
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Also, if we are bumping the version, we might need to bump it as well on pypi. cc @dianaml0
@@ -1 +1 @@ | |||
0.0.14.dev | |||
0.0.15.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would have preferred if we bumped the version in a follow-up PR
[ghstack-poisoned]
ghstack-source-id: 3d0c86b5ac9297347e0a40fd20f83dfec02f74ae Pull Request resolved: #518
[ghstack-poisoned]
ghstack-source-id: 128d2ff4077d15fa24ceae532bfc5bd743e71b31 Pull Request resolved: #518
ghstack-source-id: 128d2ff4077d15fa24ceae532bfc5bd743e71b31 Pull Request resolved: #518
Stack from ghstack (oldest at bottom):