Skip to content

Gunicorn debug mode#1190

Merged
asvetlov merged 8 commits intoaio-libs:1.0from
imbolc:gunicorn_debug_mode
Sep 20, 2016
Merged

Gunicorn debug mode#1190
asvetlov merged 8 commits intoaio-libs:1.0from
imbolc:gunicorn_debug_mode

Conversation

@imbolc
Copy link
Contributor

@imbolc imbolc commented Sep 19, 2016

What do these changes do?

It just removes debug setting from gunicorn worker wich cause error when it differs from app.debug

Copy link
Member

@asvetlov asvetlov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense but please fix tests first.

@imbolc
Copy link
Contributor Author

imbolc commented Sep 19, 2016

Sorry, tests fixed.

@asvetlov
Copy link
Member

Would you update CHANGES as well?

@imbolc
Copy link
Contributor Author

imbolc commented Sep 19, 2016

Done

@asvetlov
Copy link
Member

Looks like the robot still don't want to accept your PR 😄

@codecov-io
Copy link

codecov-io commented Sep 19, 2016

Current coverage is 98.34% (diff: 100%)

Merging #1190 into 1.0 will decrease coverage by <.01%

@@                1.0      #1190   diff @@
==========================================
  Files            29         29          
  Lines          6534       6529     -5   
  Methods           0          0          
  Messages          0          0          
  Branches       1095       1094     -1   
==========================================
- Hits           6426       6421     -5   
  Misses           56         56          
  Partials         52         52          

Powered by Codecov. Last update bd1acfc...1bde8a4

@imbolc
Copy link
Contributor Author

imbolc commented Sep 19, 2016

Merge it, before they change their mind :)

Copy link
Member

@asvetlov asvetlov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@asvetlov asvetlov merged commit e640d5e into aio-libs:1.0 Sep 20, 2016
@lock
Copy link

lock bot commented Oct 29, 2019

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.

@lock lock bot added the outdated label Oct 29, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants