Skip to content
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

Add allow_redirects option #858

Merged
merged 2 commits into from Aug 19, 2018
Merged

Add allow_redirects option #858

merged 2 commits into from Aug 19, 2018

Conversation

djstein
Copy link

@djstein djstein commented Aug 7, 2018

Fixes #830

Latest release v1.40 has broken get_archive_link
#830

@sfdye
Copy link
Member

sfdye commented Aug 10, 2018

Would you mind removing the diff introduced by the linter, so we can focus the purpose of this PR?

@Krafty-Coder
Copy link

This was a long code to review and not sure where you were to work on. The linter made the diff longer which I don't think it was the purpose of the PR.

@sfdye
Copy link
Member

sfdye commented Aug 15, 2018

@djstein I would like to make a new release, if we can get this merged.

@sfdye sfdye changed the title allow_redirects=False #830 Add allow_redirects option Aug 15, 2018
@mfonville
Copy link
Contributor

Only the changes on line 107 and 133 are relevant; @sfdye I think as maintainer I believe you are able to edit the PR.

@sfdye
Copy link
Member

sfdye commented Aug 15, 2018

@mfonville Yeah technically I can, but I don't want to modify someone else's branch without asking them first. So @djstein do you mind?

@djstein
Copy link
Author

djstein commented Aug 17, 2018

@sfdye auto formatting has been removed

@sfdye sfdye merged commit 43d325a into PyGithub:master Aug 19, 2018
@djstein
Copy link
Author

djstein commented Aug 22, 2018

@sfdye when do we see a build of this going out?

@sfdye
Copy link
Member

sfdye commented Aug 23, 2018

Until #872 progresses, my hands are tied 😞

@sfdye
Copy link
Member

sfdye commented Sep 8, 2018

1.43 is out. pip install PyGithub --upgrade

candrikos pushed a commit to candrikos/PyGithub that referenced this pull request Sep 25, 2020
Fixes PyGithub#830

Latest release v1.40 has broken get_archive_link
PyGithub#830
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants