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

Reply via email to