Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

State minimum version for gh and remove conditional code #1088

Merged
merged 4 commits into from
Apr 3, 2020
Merged

Conversation

hadley
Copy link
Member

@hadley hadley commented Apr 3, 2020

No description provided.

@DavisVaughan
Copy link
Member

Closes #1087

@hadley hadley requested a review from DavisVaughan April 3, 2020 15:15
@jennybc
Copy link
Member

jennybc commented Apr 3, 2020

There is at least one other place where we wrapped a gh call due to versions, in the pr_*() family. Let me find that and undo it here as well, if we're going to increase gh min version.

@jennybc
Copy link
Member

jennybc commented Apr 3, 2020

Yeah it's #991

jennybc added 3 commits April 3, 2020 08:58
Zero-length character might be a better return value but downstream code seems to rely on this NULL and I don't feel like pulling this thread
@jennybc jennybc changed the title Remove gh wrapper now that it's not needed State minimum version for gh and remove conditional code Apr 3, 2020
@jennybc jennybc merged commit a132195 into master Apr 3, 2020
@jennybc jennybc deleted the gh branch April 16, 2020 18:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants