-
Notifications
You must be signed in to change notification settings - Fork 14
r1.4 #167
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
r1.4 #167
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.
Thanks a lot Fernando
documentation/API_documentation/kyc-fill-in-API-Readiness-Checklist.md
Outdated
Show resolved
Hide resolved
documentation/API_documentation/kyc-match-API-Readiness-Checklist.md
Outdated
Show resolved
Hide resolved
…klist.md Co-authored-by: Herbert Damker <[email protected]>
…ist.md Co-authored-by: Herbert Damker <[email protected]>
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.
Thanks
Hi @camaraproject/release-management_maintainers please take a quick look to check that everything is correct. |
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.
Thank you, @fernandopradocabrillo .
Hi @hdamker, @camaraproject/release-management_maintainers is it okey if we merge this PR? Thanks! |
Sorry, I was ooo last week. Have created now the review issue in RM. |
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.
Looks from release management perspective.
One suggestion for clarity, which you can take or leave.
Co-authored-by: Herbert Damker <[email protected]>
1bf895b
I'll take it, I think it provides detail |
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. Thank you.
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.
Thanks
What type of PR is this?
Add one of the following kinds:
What this PR does / why we need it:
Preparation for public release r1.4. It includes updates and fixes for test plan to be fully compliant with version v0.2.1 and a correction in the API Readiness checklist.
Which issue(s) this PR fixes:
Fixes #159