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

longrun builkdite: fix dt for mhs; add mo test for mhs #833

Merged
merged 1 commit into from
Sep 9, 2022

Conversation

jiahe23
Copy link
Contributor

@jiahe23 jiahe23 commented Sep 9, 2022

PULL REQUEST

Purpose and Content

(Clearly and concisely state the purpose of this PR)

Benefits and Risks

(State concisely the benefits to be derived from and the risks associated with this PR)

Linked Issues

(Provide references to any link issues. Use closes #issuenum to automatically close an open issue)

  • Fixes #
  • Closes #

PR Checklist

  • This PR has a corresponding issue OR is linked to an SDI.
  • I have followed CliMA's codebase contribution and style guidelines OR N/A.
  • I have followed CliMA's documentation policy.
  • I have checked all issues and PRs and I certify that this PR does not duplicate an open PR.
  • I linted my code on my local machine prior to submission OR N/A.
  • Unit tests are included OR N/A.
  • Code used in an integration test OR N/A.
  • All tests ran successfully on my local machine OR N/A.
  • All classes, modules, and function contain docstrings OR N/A.
  • Documentation has been added/updated OR N/A.

@jiahe23 jiahe23 marked this pull request as ready for review September 9, 2022 20:06
@jiahe23
Copy link
Contributor Author

jiahe23 commented Sep 9, 2022

bors try

bors bot added a commit that referenced this pull request Sep 9, 2022
@bors
Copy link
Contributor

bors bot commented Sep 9, 2022

try

Build failed:

@jiahe23
Copy link
Contributor Author

jiahe23 commented Sep 9, 2022

bors try

bors bot added a commit that referenced this pull request Sep 9, 2022
@jiahe23
Copy link
Contributor Author

jiahe23 commented Sep 9, 2022

bors try-

@jiahe23
Copy link
Contributor Author

jiahe23 commented Sep 9, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Sep 9, 2022

Build succeeded:

@bors bors bot merged commit 8cf33ab into main Sep 9, 2022
@bors bors bot deleted the jh/longrun-mhs-dt branch September 9, 2022 23:45
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.

1 participant