On Fri, Feb 10, 2017 at 1:46 AM, Brett Cannon <brett@python.org> wrote:
As of right now Senthil is (I'm assuming) generating another test repo right now to see the results. If we're happy with the output then we will go with it, else we will skip the rewrite. So you're not too late as in a final decision has been made, but then you could argue you're too early based on how this test goes. :)
No need to wait, I put together a script that shows the result of the rewriting :) The script checks against roundup, so invalid ids are not converted. I'm currently tweaking it as I find issues, so it's still a bit messy, but this should give you a good idea. Drop the attached script in the root of your cpython clone and run it with: python convcm.py (to see 100 random converted commit messages) python convcm.py csid1 csid2 (to see the specified changeset ids) The script will create an output.html file that shows the before/after. It will also cache the valid ids in a valid_ids.json files. There are still a few ambiguous cases, in particular: * I left "SF", so "SF issue #12345" becomes "SF bpo-12345"; * I left "patch", so "Apply patch #12345" becomes "Apply patch bpo-12345"; Senthil is also testing the conversion using this regex. Best Regards, Ezio Melotti