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

Release notes O3DE 23.05.0 #2367

Merged
merged 17 commits into from
May 2, 2023

Conversation

chanmosq
Copy link
Contributor

Change summary

Submission Checklist:

  • Descriptive active voice - Do descriptive sentences have a clear subject and action verb?
  • Answer the question at hand - Does the documentation answer a what, why, how, or where type of question?
  • Consistency - Does the content consistently follow the Style Guide?
  • Help the user - Does the documentation show the user something meaningful?

Signed-off-by: chanmosq <75444793+chanmosq@users.noreply.github.com>
@chanmosq chanmosq requested review from a team as code owners April 17, 2023 22:32
@chanmosq chanmosq added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 17, 2023
Signed-off-by: chanmosq <75444793+chanmosq@users.noreply.github.com>
Copy link

@tonybalandiuk tonybalandiuk left a comment

Choose a reason for hiding this comment

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

Left comments. A couple of them refer to updating these release notes with the current content from the feature list.

@chanmosq chanmosq marked this pull request as draft April 20, 2023 18:09
chmosquera and others added 3 commits April 22, 2023 18:38
Signed-off-by: Chanelle Mosquera <chmosquera@users.noreply.github.com>
Signed-off-by: chanmosq <75444793+chanmosq@users.noreply.github.com>
Signed-off-by: chanmosq <75444793+chanmosq@users.noreply.github.com>
@chanmosq chanmosq marked this pull request as ready for review April 24, 2023 20:12
Signed-off-by: chanmosq <75444793+chanmosq@users.noreply.github.com>
@chanmosq chanmosq removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 25, 2023
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
@chanmosq
Copy link
Contributor Author

Reformatting the release notes in SIG Release first: o3de/sig-release#182
Then I will migrate them over here.

Signed-off-by: chanmosq <75444793+chanmosq@users.noreply.github.com>
Copy link

@tonybalandiuk tonybalandiuk left a comment

Choose a reason for hiding this comment

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

approved. Left a comment. We might be able to improve how we talk about ROS2 but it's not critical.

Copy link
Contributor

@micronAMZN micronAMZN left a comment

Choose a reason for hiding this comment

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

Applying several suggestions

content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
content/docs/release-notes/2305-0-release-notes.md Outdated Show resolved Hide resolved
Signed-off-by: Mike Cronin <58789750+micronAMZN@users.noreply.github.com>
Signed-off-by: Mike Cronin <58789750+micronAMZN@users.noreply.github.com>
Signed-off-by: Mike Cronin <58789750+micronAMZN@users.noreply.github.com>
Signed-off-by: Mike Cronin <58789750+micronAMZN@users.noreply.github.com>
Signed-off-by: Mike Cronin <58789750+micronAMZN@users.noreply.github.com>
Signed-off-by: Mike Cronin <58789750+micronAMZN@users.noreply.github.com>
Signed-off-by: Mike Cronin <58789750+micronAMZN@users.noreply.github.com>
Signed-off-by: Mike Cronin <58789750+micronAMZN@users.noreply.github.com>
Signed-off-by: Mike Cronin <58789750+micronAMZN@users.noreply.github.com>
Signed-off-by: Mike Cronin <58789750+micronAMZN@users.noreply.github.com>
@chanmosq chanmosq merged commit 933781b into o3de:development May 2, 2023
@chanmosq chanmosq deleted the 2305-release-notes branch May 2, 2023 17:52
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