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

2023 09 15 release #6890

Merged
merged 3 commits into from
Sep 15, 2023
Merged

2023 09 15 release #6890

merged 3 commits into from
Sep 15, 2023

Conversation

mwu2018
Copy link
Contributor

@mwu2018 mwu2018 commented Sep 15, 2023

What this PR does

Fixes #

This is a description of what I've done in this PR, especially explaining choices made to make the reviewer's job easier. If I don't replace this paragraph with an actual description, my PR will probably be ignored.

Test me

How should reviewers test this? (Hint: If you want to provide a test catalog item, create a Gist of its catalog JSON, add its url and your branch name to this url: http://ci.terria.io/<branch name>/#clean&<raw url of your gist> , and then paste that in the body of this PR)

Checklist

  • There are unit tests to verify my changes are correct or unit tests aren't applicable (if so, write quick reason why unit tests don't exist)
  • I've updated relevant documentation in doc/.
  • I've updated CHANGES.md with what I changed.
  • I've provided instructions in the PR description on how to test this PR.

Copy link
Contributor

@tephenavies tephenavies left a comment

Choose a reason for hiding this comment

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

Fixed a changelog entry being under the wrong version (found with comparing 8.3.3 to this branch and confirmed that it didn't make 8.3.3 using git history).

This is good to release.

@mwu2018 mwu2018 merged commit 5e41ba0 into main Sep 15, 2023
@mwu2018 mwu2018 deleted the 2023-09-15-release branch September 15, 2023 05:02
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.

2 participants