-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add module_name variable #2
Conversation
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.
Yeah we can probably just do a ref to an old tag of griddataformats, that should work fine.
Co-authored-by: Irfan Alibay <[email protected]>
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.
Odd, not sure why CI isn't getting triggered.
O no it is.. invalid workflow file. |
Just undrafting to see if that's what is blocking the action. |
I've added a sparse checkout to try to get out of a bit of a cyclic issue, but it turns out it still won't trigger on PR. I'm going to merge this in the hope that there's some indication as to what's going on. |
Took a bit of work and a new token but it works now, thanks @lilyminium ! |
Fixes #1
Probably need tests for griddata/formats too...