Skip to content
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

Add support for submit_requirements fields in ChangeInfo. #169

Merged
merged 1 commit into from
Oct 20, 2024

Conversation

simony-gke
Copy link
Contributor

@simony-gke simony-gke commented Oct 11, 2024

As Gerrit had deprecated label functions and migrate to submit-requirements (https://gerrit-review.googlesource.com/Documentation/config-labels.html), our project (Prow, https://docs.prow.k8s.io/docs/overview/) needs to get this submit-requirements info associated with a change.

@simony-gke simony-gke changed the title Add support for submit_requirement fields in ChangeInfo. Add support for submit_requirements fields in ChangeInfo. Oct 11, 2024
@simony-gke
Copy link
Contributor Author

@andygrunwald Please review, thanks!

@andygrunwald
Copy link
Owner

Thanks @simony-gke for the PR.
I will review it in the upcoming week.

@simony-gke
Copy link
Contributor Author

@andygrunwald friendly ping. We're waiting on this, thanks.

@andygrunwald
Copy link
Owner

andygrunwald commented Oct 18, 2024 via email

Copy link
Owner

@andygrunwald andygrunwald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. This is looking good.

@andygrunwald andygrunwald merged commit 18c7753 into andygrunwald:master Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants