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

Bug fix regarding https://github.com/COSIMA/access-om2/issues/282 #386

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

hakaseh
Copy link

@hakaseh hakaseh commented May 31, 2024

COSIMA/access-om2/issues/282
@dougiesquire did I create a pull request in the right place??

@dougiesquire
Copy link
Collaborator

Thanks @hakaseh. Yes, I'll try to get to a review soon (I'd like to confirm for myself one last time that this is needed)

Copy link
Collaborator

@dougiesquire dougiesquire left a comment

Choose a reason for hiding this comment

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

Thanks @hakaseh - looks good to me!

@dougiesquire
Copy link
Collaborator

@aekiss, @aidanheerdegen would one of you mind (reviewing and) merging?

@aekiss
Copy link
Collaborator

aekiss commented Jun 7, 2024

Thanks @hakaseh, @dougiesquire I'm happy to merge since this looks ok to you both.

@aekiss aekiss merged commit 59e0aad into mom-ocean:master Jun 7, 2024
9 checks passed
@pwongpan
Copy link

Thanks again all.

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.

4 participants