Skip to content
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

Merged
merged 29 commits into from
Nov 5, 2024

Conversation

akoshunyadi
Copy link
Collaborator

What type of PR is this?

  • subproject management

What this PR does / why we need it:

Prepare patch release r1.3

  • update the modified OAS files (compared to r1.2)
  • update change log

Which issue(s) this PR fixes:

Special notes for reviewers:

Changelog input

Additional documentation

CHANGELOG.md Outdated Show resolved Hide resolved
@akoshunyadi akoshunyadi requested a review from a team October 10, 2024 06:57
bigludo7
bigludo7 previously approved these changes Oct 10, 2024
Copy link
Collaborator

@bigludo7 bigludo7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@hdamker hdamker left a 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.

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
akoshunyadi and others added 2 commits October 10, 2024 20:54
Co-authored-by: Herbert Damker <[email protected]>
Co-authored-by: Herbert Damker <[email protected]>
@akoshunyadi akoshunyadi requested a review from hdamker October 10, 2024 18:59
@akoshunyadi
Copy link
Collaborator Author

Review findings done, the review by https://github.com/orgs/camaraproject/teams/release-management_maintainers was already requested

@akoshunyadi akoshunyadi requested a review from a team October 10, 2024 21:18
@akoshunyadi akoshunyadi dismissed hdamker’s stale review October 14, 2024 16:26

Requested changes have been applied, Github still not satisfied

sachinvodafone
sachinvodafone previously approved these changes Oct 17, 2024
@akoshunyadi
Copy link
Collaborator Author

@camaraproject/release-management_maintainers kindly asking for final review

@tanjadegroot
Copy link

In addition to the PR comments, I saw some further points:

README.me:

  • line 10: "API family" --> "APIs"
  • line 14: "customer" --> "API consumer"
  • line 15: "has lost connectivity" --> "is not connected"

bigludo7
bigludo7 previously approved these changes Oct 31, 2024
Copy link
Collaborator

@bigludo7 bigludo7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@akoshunyadi
Copy link
Collaborator Author

@bigludo7 now it should be perfect 🙂

Copy link
Collaborator

@bigludo7 bigludo7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :)

@tanjadegroot
Copy link

Great ! OK from Release Management /LGTM

@akoshunyadi akoshunyadi merged commit a115a00 into camaraproject:main Nov 5, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants