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

Change max_redirects to max_redirect to be consistent with LWP::UserAgent #12

Merged
merged 1 commit into from Sep 28, 2015

Conversation

Projects
None yet
2 participants
@ghost

ghost commented Sep 19, 2015

No description provided.

Vincent Lequertier
Change max_redirects to max_redirect to be consistent with LWP::UserA…
…gent

Cleanup alignement

add a test unit for max_redirects and alias max_redirects to max_redirect
@kaoru

This comment has been minimized.

Show comment
Hide comment
@kaoru

kaoru Sep 28, 2015

Collaborator

Looking good :-) @evdb do you want to merge this and do a release to CPAN or do you want me to take care of it?

Collaborator

kaoru commented Sep 28, 2015

Looking good :-) @evdb do you want to merge this and do a release to CPAN or do you want me to take care of it?

@evdb

This comment has been minimized.

Show comment
Hide comment
@evdb

evdb Sep 28, 2015

Owner

All your Alex. Thanks for the improvements Vincent.

Owner

evdb commented Sep 28, 2015

All your Alex. Thanks for the improvements Vincent.

kaoru added a commit that referenced this pull request Sep 28, 2015

Merge pull request #12 from SkySymbol/use_max_redirect
Change max_redirects to max_redirect to be consistent with LWP::UserAgent

@kaoru kaoru merged commit f8c4d8e into evdb:master Sep 28, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@ghost ghost deleted the use_max_redirect branch Sep 28, 2015

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Sep 28, 2015

You're welcome :-)

ghost commented Sep 28, 2015

You're welcome :-)

@kaoru

This comment has been minimized.

Show comment
Hide comment
@kaoru

kaoru Sep 28, 2015

Collaborator

@SkySymbol HTTP-Async 0.30 with your changes is on its way to CPAN 👍 😄

Collaborator

kaoru commented Sep 28, 2015

@SkySymbol HTTP-Async 0.30 with your changes is on its way to CPAN 👍 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment