Mathew Robinson
Mathew Robinson
Hi @piptouque thanks for the contribution! I'd like to see some new tests for this behaviour. The code is a bit obtuse and I have some potential style improvements but...
Sounds good to me, I'll get you a PR. Thanks!
@piptouque this PR is quickly spiraling into a rewrite. I have no intention of merging the use of `GitPython`. I intentionally didn't use it because keeping a low dependency count...
Will write unit tests later today and am signing the CLA now.
Example of bad JSON before this change: ``` { "name": " market, the Company logo returns the user to the home page [UK]", "status": "passed", "steps": [ { "name": "Given...
So I've been working through the broken tests but would like some confirmation that this solution will be acceptable before I spend much more time on this if that's possible?
@aspenboy @JagadeeshJayachandran while I appreciate the reviews, AFAICT the maintainer of this repository is @sseliverstov and they haven't merged any PRs in quite a long time. I will not be...
I guess bump to stop the stale bot?
I'd be willing to put a PR up for this if there was acknowledgement that it would get accepted as a valid change. Just don't wanna go through the trouble...
This is affecting us with our database migrations (which we currently create as Jobs). I'd be interested in fixing this if there was a clear direction on what solution has...