-
Notifications
You must be signed in to change notification settings - Fork 0
Less restrictive target for gh
#17
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
Conversation
|
🚀 Attempted to deploy 🖥️
|
|
!redeploy |
|
🚀 Attempted to deploy 🖥️
|
|
This works on normalbw now: |
|
The model version in the
|
|
🚀 Attempted to deploy 🖥️
|
|
The model version in the
|
|
🚀 Attempted to deploy 🖥️
|
|
The model version in the
|
|
!bump minor |
|
🚀 Attempted to deploy 🖥️
|
|
🚀 Attempted to deploy 🖥️
|
Hmm. I think the spack environment names would clash, unfortunately. Maybe it's worth doing a small version update with these...? |
|
🚀 Attempted to deploy 🖥️
|
|
🚀 Attempted to deploy 🖥️
|
As we're still in spack/0.22 there aren't newer versions available for these packages: |
|
We can change the module projections (e.g. append |
|
Correct, environments are set statically in the CI. This problem won't be the case once we move to spack:
definitions:
- _name: [gh]
- _version: [2.43.1-1] # Note the extra `-1`
specs:
- [email protected] |
|
Thanks - what can we do in the interim. This fixes a kind of annoying bug in our recomended way of authenticating to git |
|
I've a PR to update the spack instance to enable newer versions of As noted in that PR
So @anton-seaice, once we have the PR above merged can you remove |
|
The instances of spack have been updated to include the new versions of |
This reverts commit a19608f.
|
🚀 Attempted to deploy 🖥️
|
|
The model version in the
|
|
🚀 Attempted to deploy 🖥️
|
|
🚀 Attempted to deploy 🖥️
|
|
The model version in the
|
|
🚀 Attempted to deploy 🖥️
|
|
🚀 Attempted to deploy 🖥️
|
|
I think this is ready @aidanheerdegen |
CodeGat
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm no Aidan, but I think this looks good
|
Please squash and merge @anton-seaice. |
Contibutes to #15
🚀 The latest prerelease
pinentry/pr17-5at a19608f is here: #17 (comment) 🚀🚀 The latest prerelease
gh/pr17-8at aa771b0 is here: #17 (comment) 🚀