psss / did

What did you do last week, month, year?
https://did.readthedocs.io/
GNU General Public License v2.0
247 stars 105 forks source link

Add searching for work in progress Gerrit changes #187

Closed hferreiro closed 5 years ago

hferreiro commented 5 years ago

Add support for searching CLs marked as "Work in progress". These are CLs for which the code review hasn't started yet. The submitted CLs report is changed to omit these WIP CLs. Additionally, remove support for drafts ('is:draft') which isn't available in Gerrit anymore and probably had the same purpose.

hferreiro commented 5 years ago

I'm not sure what's the purpose of limit_since. Can anyone clarify?

psss commented 5 years ago

Looks good. Thanks for the patch. Regarding the limit_since parameter: I don't know all the implementation details of the gerrit plugin. It was originally written by @mcepl. Maybe he might have some hint...