Stephen Finucane <step...@that.guru> writes: > There are two scripts provided in tools that reference the removed > hash function of the 'parser' script. Move this hash function back into > its own module and update these scripts to use said module.
So, we need to start doing tests of this stuff - even if it's a separate test script that doesn't integrate nicely into the python tests. I'm happy to take this on - if you're happy to wait until the new year. Regards, Daniel > > Changes since v2: > - Only move the hash function into its own module > - Don't use argparse as it's not really needed > > Stephen Finucane (5): > parser: Remove unused parameter > parser: Trivial rename of functions > hasher: Create hasher module > tools: Trivial formatting fixes > tools: Update to use 'hasher' > > patchwork/hasher.py | 87 +++++++++++++++++++++++++++++++ > patchwork/models.py | 52 +----------------- > patchwork/parser.py | 14 +++-- > patchwork/tests/test_parser.py | 31 ++++------- > tools/patchwork-update-commits | 33 +++++++++--- > tools/post-receive.hook | 116 > ++++++++++++++++++++++++----------------- > 6 files changed, 199 insertions(+), 134 deletions(-) > create mode 100644 patchwork/hasher.py > > -- > 2.9.3 > > _______________________________________________ > Patchwork mailing list > Patchwork@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/patchwork _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork
