[Twisted-Python] twisted.web with dynamic + static content

Glyph glyph at twistedmatrix.com
Tue Nov 27 22:53:13 MST 2018



> On Nov 27, 2018, at 2:37 PM, Jeff Grimmett <grimmtooth at gmail.com> wrote:
> 
> Thanks, Tom.  
> 
> Is there a limit to the smallness of the proposed changes?  For example, if I had a possible two-line change to https://twistedmatrix.com/documents/current/api/twisted.internet.wxreactor.html <https://twistedmatrix.com/documents/current/api/twisted.internet.wxreactor.html> (which looks like it would be done in the docstring), is it okay to just have a PR for that or does it need to be bundled with more changes?
> 
> Thanks,
> 
> Jeff 

Smaller changes are always better!  Sometimes changes need to be larger and don't make sense when split up, so it's OK to propose large things, but a one-liner is the easiest possible thing to review, and will generally get integrated faster.

-g
-------------- next part --------------
An HTML attachment was scrubbed...
URL: </pipermail/twisted-python/attachments/20181127/a3dc3748/attachment-0002.html>


More information about the Twisted-Python mailing list