Skip to content

Instantly share code, notes, and snippets.

@adiroiban
Last active May 3, 2016 07:50
Show Gist options
  • Save adiroiban/0adaf7824b93a14293580d94c4ac4f23 to your computer and use it in GitHub Desktop.
Save adiroiban/0adaf7824b93a14293580d94c4ac4f23 to your computer and use it in GitHub Desktop.
Twisted initial review reply
Many thanks for your contribution.
Can you please define the scope of your patch or update the ticket's description?
DETAILS ABOUT WHY THE SCOPE IS NOT OK.
The patch needs at least one automated test to demonstrate the change and to prevent future regressions.
The patch also needs a NEWS file fragment (release notes fragment)
as described here http://twistedmatrix.com/trac/wiki/ReviewProcess#Newsfiles
If this is useful for "production" usage,
then the news fragment should describe how it can be useful...
so that in the next release people are aware of this new thing :)
It would be nice if the new thing is also documented so that people will know how to use this new nice thing.
Please consider my comments and submit a new patch.
Thanks again!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment