- Notifications
You must be signed in to change notification settings - Fork786
Add add label to an issue#461
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
4fbc7bd to81706baCompare81706ba tof9db695Compare
mtscout6 left a comment
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.
Please add tests for these.
asmsuechan commentedJul 19, 2017
Sure |
j-rewerts commentedApr 7, 2019
@clayreimann This API is marked as preview by GitHub. What is the policy on using preview APIs? |
clayreimann commentedApr 9, 2019
@j-rewerts it's unclear. I guess we could mark the API as provisional in this library and let clients assume the risks |
j-rewerts commentedApr 9, 2019
Yeah that's a good idea. I'll take this code and write tests for them when I have a second. I've added an issuehere for going through the code base and adding warnings like this. |
clayreimann commentedApr 9, 2019
Sounds good |
I could not find these methods so I created. If anyone has any information about them, please let me know.