-
Notifications
You must be signed in to change notification settings - Fork 9
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
Public Release for Fall24 meta-release #178
Conversation
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 - Thanks @PedroDiez
Up to @hdamker and the release Management team ;)
@PedroDiez If you want to have the meeting soon within main, you may want to do it in an own PR. Could take some days before the release PR gets approved by the RM team. |
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.
- API Readiness checklists should be updated to r1.2 versions (without -rc suffixes), and resolve any pending tbd
Thanks Herbert, I will comment tomorrow in the WG meeting. I think we can wait because relevant people is already aware of new schedule |
documentation/API_documentation/carrier-billing-refund-API-Readiness-Checklist.md
Outdated
Show resolved
Hide resolved
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.
Some projects are naming User stories with the same format than API-Readiness-Checklist, that is api-name-User-Story.md, but it is not made a requirement
…diness-Checklist.md Co-authored-by: Jose Luis Urien <jlurien@gmail.com>
Thanks @jlurien I am applying it, It is fine to align |
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.
Approved on behalf of Release Management
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
After TSC meeting, it can be merged: SC Approval of M4 (all release PRs of Sub Project reviewed by Release Management)
|
What type of PR is this?
Add one of the following kinds:
What this PR does / why we need it:
PR to generate Public Release of Carrier Billing Checkout Family APIs
Which issue(s) this PR fixes:
Fixex #155
Fixes #168
Fixes #173
Fixes #175
Special notes for reviewers:
PRs #174 and #176 to be merged first than this one
Changes driven in API Spec YAMLs:
confirmPayment
operationChangelog input
Additional documentation
N/A