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

Re-pin ASV again and list env info #613

Merged
merged 3 commits into from
May 3, 2024
Merged

Conversation

kafitzgerald
Copy link
Contributor

@kafitzgerald kafitzgerald commented May 3, 2024

PR Summary

Re-pins ASV again and lists environment info

Related Tickets & Documents

Relates to #555

PR Checklist

General

  • PR includes a summary of changes
  • Link relevant issues, make one if none exist
  • Add a brief summary of changes to docs/release-notes.rst in a relevant section for the upcoming release.
  • Add appropriate labels to this PR
  • PR follows the Contributor's Guide

@kafitzgerald kafitzgerald added dependencies Pull requests that update a dependency file scalability and performance Related to scalability, performance, or benchmarking labels May 3, 2024
@kafitzgerald
Copy link
Contributor Author

This still fails on my fork because the ASV version pin needs to be merged into main, but listing the env info, etc. is all working fine.

@kafitzgerald kafitzgerald marked this pull request as ready for review May 3, 2024 17:28
@anissa111 anissa111 merged commit 9663008 into NCAR:main May 3, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file scalability and performance Related to scalability, performance, or benchmarking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants