CarrierBillingCheckOut r1.2 release review (M4)

[hdamker]

Release PR or issue to review

camaraproject/CarrierBillingCheckOut#178

Review actions

Assign this issue to yourself and follow the below list.
For any review action, review the file(s) in the issue/PR listed above.
Put comments in the above issue or PR if they concern non-changed files/text.
Put a short summary of the main review result here into the review issue.

• API definition files (YAML) (check version in info & servers objects)
• test file(s) availability
• changelog updated
• readme updated (enforce correct release number / API version naming)
• API readiness checklist(s)

Release actions

Assign this issue to yourself or another RM team member and follow the below list.
When done, tick the box in this issue.

• API release tracker available on wiki
• Review comments addressed (by Sub Project)
• Release PR approved (on behalf of Release Management)
• PR merged (by Sub Project codeowner)
• Release created within GitHub (by Sub Project codeowner)
• Release Tracker updated (with creation date of the release and the release tag link)

Additional comments

[jlurien]

@hdamker it seems I cannot edit the issue above to mark the validated tasks, am I missing anything?

[jlurien]

• Test files not part of the PR, but already merged in main
• Some APIs are adding "- rc" to "r1.1" in the Changelog, while other don't. There are no explicit instructions about this but it is not consistent across the project
• API readiness checklists not updated to lastest versions

[hdamker]

@hdamker it seems I cannot edit the issue above to mark the validated tasks, am I missing anything?

Yes, write access to the repository 😃 ... fixed now, have granted all reviewers write access.

[jlurien]

Review actions completed

[jlurien]

All task completed

[tanjadegroot]

Thanks @jlurien for the RM review. This is can be closed.