-
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
Fix issue with build and deploy #128
Conversation
ℹ️ Modified WorkflowsThis pull request contains modified workflow files and no preview will be created. Workflow files modified:
If this is not from a trusted source, please inspect the changes for any malicious content. |
This should fix the GH action failure: https://github.com/esciencecenter-digital-skills/geospatial-python/actions/runs/12759005340 |
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.
Looks good! Thanks! sorry overlooked when reviewing #129.
Just one small remark, maybe let's document this pinning somewhere (like an issue) and revisit this later (in a year)? Maybe ubuntun 24 machine will have R at that time.
Auto-generated via `{sandpaper}` Source : 856842c Branch : main Author : Francesco Nattino <[email protected]> Time : 2025-01-15 10:04:34 +0000 Message : Merge pull request #128 from esciencecenter-digital-skills/fix-build-and-deploy Fix issue with build and deploy
Auto-generated via `{sandpaper}` Source : a3471f8 Branch : md-outputs Author : GitHub Actions <[email protected]> Time : 2025-01-15 10:06:15 +0000 Message : markdown source builds Auto-generated via `{sandpaper}` Source : 856842c Branch : main Author : Francesco Nattino <[email protected]> Time : 2025-01-15 10:04:34 +0000 Message : Merge pull request #128 from esciencecenter-digital-skills/fix-build-and-deploy Fix issue with build and deploy
Auto-generated via `{sandpaper}` Source : a3471f8 Branch : md-outputs Author : GitHub Actions <[email protected]> Time : 2025-01-15 10:06:15 +0000 Message : markdown source builds Auto-generated via `{sandpaper}` Source : 856842c Branch : main Author : Francesco Nattino <[email protected]> Time : 2025-01-15 10:04:34 +0000 Message : Merge pull request #128 from esciencecenter-digital-skills/fix-build-and-deploy Fix issue with build and deploy
Auto-generated via `{sandpaper}` Source : a3471f8 Branch : md-outputs Author : GitHub Actions <[email protected]> Time : 2025-01-15 10:06:15 +0000 Message : markdown source builds Auto-generated via `{sandpaper}` Source : 856842c Branch : main Author : Francesco Nattino <[email protected]> Time : 2025-01-15 10:04:34 +0000 Message : Merge pull request #128 from esciencecenter-digital-skills/fix-build-and-deploy Fix issue with build and deploy
Ubuntu 24 comes with no R installed, fix issues with GH action by pinning Ubuntu 22.
Applying this fix: https://github.com/carpentries/sandpaper/pull/606/files
See issue: carpentries/sandpaper#605