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

Pull Request #3

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Pull Request #3

wants to merge 3 commits into from

Conversation

djeanprost
Copy link

Hello,

I added the possibility to setup http header in response. Can you review this and maybe accept it ?
Regards,
Dom

@jharlap
Copy link
Owner

jharlap commented Oct 1, 2012

Unfortunately you reformatted the code, so a normal diff (such as displayed by github) makes it seem like the whole files have changes rather than just your edits. If you correct that, I can review it easily. Otherwise, it'll have to wait until I have time to pull your code and go over it in detail.

Cheers,
J

@djeanprost
Copy link
Author

Hello Jonathan,

Indeed, I did not think to this problem.
Anyway, Ifixed my code style so that you should be able to review the changes.
Hope it's ok this time.
dom

# 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.

2 participants