-
-
Notifications
You must be signed in to change notification settings - Fork 188
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
remove legacy jekyll template inclusion #341
remove legacy jekyll template inclusion #341
Conversation
🆗 Pre-flight checks passed 😃This pull request has been checked and contains no modified workflow files, spoofing, or invalid commits. It should be safe to Approve and Run the workflows that need maintainer approval. |
Hi @cpauvert, thanks for your first pr! And sorry for the slow reply - @vhmcck and I are new maintainers and it has taken us a little while to get up to speed. And doing:
is indeed a good way to check they lesson is building okay. Thanks again @cpauvert ! |
Thank you for considering this : ) And good luck on your maintainers path! |
Auto-generated via {sandpaper} Source : 394a20d Branch : main Author : Paul Smith <[email protected]> Time : 2023-08-14 08:06:09 +0000 Message : Merge pull request #341 from cpauvert/fix-340-legacy-gh-variables remove legacy jekyll template inclusion
Auto-generated via {sandpaper} Source : 35fdf61 Branch : md-outputs Author : GitHub Actions <[email protected]> Time : 2023-08-14 08:08:53 +0000 Message : markdown source builds Auto-generated via {sandpaper} Source : 394a20d Branch : main Author : Paul Smith <[email protected]> Time : 2023-08-14 08:06:09 +0000 Message : Merge pull request #341 from cpauvert/fix-340-legacy-gh-variables remove legacy jekyll template inclusion
Auto-generated via {sandpaper} Source : 35fdf61 Branch : md-outputs Author : GitHub Actions <[email protected]> Time : 2023-08-14 08:08:53 +0000 Message : markdown source builds Auto-generated via {sandpaper} Source : 394a20d Branch : main Author : Paul Smith <[email protected]> Time : 2023-08-14 08:06:09 +0000 Message : Merge pull request #341 from cpauvert/fix-340-legacy-gh-variables remove legacy jekyll template inclusion
Auto-generated via {sandpaper} Source : 35fdf61 Branch : md-outputs Author : GitHub Actions <[email protected]> Time : 2023-08-14 08:08:53 +0000 Message : markdown source builds Auto-generated via {sandpaper} Source : 394a20d Branch : main Author : Paul Smith <[email protected]> Time : 2023-08-14 08:06:09 +0000 Message : Merge pull request #341 from cpauvert/fix-340-legacy-gh-variables remove legacy jekyll template inclusion
Hi!
This fixes #340.
Removing the line improves readability and does not seem to break anything else in the lesson, after using the following command with the Carpentries Workbench:
Hope that helps and let me know if I should run additional tests to be sure not to break anything.
BEst,