New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Added the ability to filter MRs by 'state' to Gitlab PR Generator #9540
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9540 +/- ##
=======================================
Coverage 45.89% 45.90%
=======================================
Files 222 222
Lines 26458 26461 +3
=======================================
+ Hits 12143 12146 +3
Misses 12658 12658
Partials 1657 1657
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Can you add a line to the docs mentioning the filter?
Simplied state verification + Added Unit test Signed-off-by: Aymen Ben Tanfous <aymen.bentanfous@cimpress.com>
Signed-off-by: Aymen Ben Tanfous <aymen.bentanfous@cimpress.com>
@crenshaw-dev would this feat and this one #9462 be release in the next release? I'm not yet familiar with the release way and cadence in argo project. |
@Aym3nTN this will be in 2.5, which is expected in August. |
…or (argoproj#9540) * Added the ability to filter MRs by 'state' to Gitlab PR Generator Simplied state verification + Added Unit test Signed-off-by: Aymen Ben Tanfous <aymen.bentanfous@cimpress.com> * Updated the doc Signed-off-by: Aymen Ben Tanfous <aymen.bentanfous@cimpress.com> Co-authored-by: Aymen Ben Tanfous <aymen.bentanfous@cimpress.com>
Signed-off-by: Aymen Ben Tanfous aymen.bentanfous@cimpress.com
Closes: #9539
Checklist: