Conversation
|
change log has not been updated yet. will do it after the other issue #15 is complete |
|
all the open issues have been closed in this repo. |
Critical Issues Requiring Immediate Attention
|
|
@urvika-v Thank you - there are no more critical issues - I'll continue the review. Quick fix: The Readiness checklist needs to be updated to the template:
|
|
@rartych Updated API-Readiness-Checklist |
|
I think the first line in the Release Information section in README.md will be outdated after r1.1 . BTW, More detailed template for this section is expected for M4. |
documentation/API_documentation/application-profiles-API-Readiness-Checklist.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Rafal Artych <121048129+rartych@users.noreply.github.com>
|
In API-Readiness-Checklist links to Commonalites and ICM release are expected: |
|
Is error relevant for this API? If so maybe the description and message can be updated - this can be discussed and addressed by the team before M4 Similarly: |
…ness-Checklist.md Co-authored-by: Rafal Artych <121048129+rartych@users.noreply.github.com>
Co-authored-by: Rafal Artych <121048129+rartych@users.noreply.github.com>
Co-authored-by: Rafal Artych <121048129+rartych@users.noreply.github.com>
|
@urvika-v Almost done! |
Co-authored-by: Rafal Artych <121048129+rartych@users.noreply.github.com>
|
Please check the README.md as indicated in #16 (comment) |
@rartych readme has been update. let me know if this looks good. |
rartych
left a comment
There was a problem hiding this comment.
Looks good from Release Management perspective
Kevsy
left a comment
There was a problem hiding this comment.
LGTM, approved as codeowner.
What type of PR is this?
Add one of the following kinds:
What this PR does / why we need it:
Creating the release candidate for Fall 25
Which issue(s) this PR fixes:
Fixes #17
Alignment with latest commonalities and ICM guidelines.
Special notes for reviewers:
Changelog input
Additional documentation
This section can be blank.