Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Test vertical plotting with a filename specified #712

Merged
merged 1 commit into from
Jul 1, 2024
Merged

Test vertical plotting with a filename specified #712

merged 1 commit into from
Jul 1, 2024

Conversation

jfrost-mo
Copy link
Member

@jfrost-mo jfrost-mo commented Jun 28, 2024 •
edited
Loading

Contribution checklist

Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.

  • Documentation has been updated to reflect change.
  • New code has tests, and affected old tests have been updated.
  • All tests and CI checks pass.
  • Ensured the pull request title is descriptive.
  • Conda lock files have been updated if dependencies have changed.
  • Attributed any Generative AI, such as GitHub Copilot, used in this PR.
  • Marked the PR as ready to review.

@jfrost-mo jfrost-mo added small Small issue that should be quick to resolve cleanup Non-functional improvement labels Jun 28, 2024
@jfrost-mo jfrost-mo self-assigned this Jun 28, 2024
@github-actions GitHub Actions
Copy link
Contributor

Coverage

@jfrost-mo jfrost-mo marked this pull request as ready for review June 28, 2024 12:59
dasha-shchep
dasha-shchep approved these changes Jul 1, 2024
View reviewed changes
Copy link
Contributor

@dasha-shchep dasha-shchep left a comment

Choose a reason for hiding this comment

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

Good test 👍

@dasha-shchep dasha-shchep merged commit 7c4d241 into main Jul 1, 2024
7 checks passed
@dasha-shchep dasha-shchep deleted the 708_vertical_plotting_tests branch July 1, 2024 09:36
@jfrost-mo jfrost-mo mentioned this pull request Jul 2, 2024
Closed
6 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@dasha-shchep dasha-shchep dasha-shchep approved these changes

Assignees

@jfrost-mo jfrost-mo

Labels
cleanup Non-functional improvement small Small issue that should be quick to resolve
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jfrost-mo @dasha-shchep