-
Notifications
You must be signed in to change notification settings - Fork 33
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
Prepare release r1.3 #216
Prepare release r1.3 #216
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
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.
@akoshunyadi please touch also shortly the API-Readiness-Checklist (first line only). And add @camaraproject/release-management_maintainers - all LGTM but as it is a public review and there need to be some updates done (e.g. the CAMARA Readme), Release Management must be involved.
Co-authored-by: Herbert Damker <herbert.damker@telekom.de>
Co-authored-by: Herbert Damker <herbert.damker@telekom.de>
Review findings done, the review by https://github.com/orgs/camaraproject/teams/release-management_maintainers was already requested |
Requested changes have been applied, Github still not satisfied
@camaraproject/release-management_maintainers kindly asking for final review |
code/API_definitions/device-reachability-status-subscriptions.yaml
Outdated
Show resolved
Hide resolved
documentation/API_documentation/device-reachability-status-API-Readiness-Checklist.md
Show resolved
Hide resolved
...tation/API_documentation/device-reachability-status-subscriptions-API-Readiness-Checklist.md
Show resolved
Hide resolved
documentation/API_documentation/device-roaming-status-subscriptions-API-Readiness-Checklist.md
Show resolved
Hide resolved
In addition to the PR comments, I saw some further points: README.me:
|
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
@bigludo7 now it should be perfect 🙂 |
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 :)
Great ! OK from Release Management /LGTM |
What type of PR is this?
What this PR does / why we need it:
Prepare patch release r1.3
Which issue(s) this PR fixes:
Special notes for reviewers:
Changelog input
Additional documentation