[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