Navigation Menu

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

Clarification of exclog.ignore documentation #27

Merged
merged 1 commit into from Sep 21, 2016
Merged

Clarification of exclog.ignore documentation #27

merged 1 commit into from Sep 21, 2016

Conversation

conradoplg
Copy link
Contributor

Changed exclog.ignore documentation to make clear it is possible to re-enable logging http exceptions.

@tseaver
Copy link
Member

tseaver commented Sep 21, 2016

@conradoplg the Python 3.2 failure is not due to this PR. I will be pushing another PR which removes 2.6 and 3.2 support, and adds 3.5 support.

@tseaver
Copy link
Member

tseaver commented Sep 21, 2016

I have restarted the build after merging #28.

@tseaver
Copy link
Member

tseaver commented Sep 21, 2016

Ugh, I guess I need to have you rebase your branch and re-push your branch. E.g.:

$ cd /path/to/your/clone/of/pyramid_exclog
$ git remote add upstream https://github.com/Pylons/pyramid_exclog.git
$ git fetch upstream
$ git rebase upstream/master
$ git push -f origin ignore-clarification

Changed exclog.ignore documentation to make clear it is possible to re-enable logging http exceptions.
@tseaver tseaver merged commit 03ae824 into Pylons:master Sep 21, 2016
@tseaver
Copy link
Member

tseaver commented Sep 21, 2016

@conradoplg Thanks for the patch!

@conradoplg conradoplg deleted the ignore-clarification branch September 21, 2016 19:57
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

2 participants