carpentries / styles

Styles for The Carpentries lessons. No README to avoid merge conflicts with lessons. Demo 👇
https://carpentries.github.io/lesson-example
Other
84 stars 96 forks source link

Template workflow: smarter syncing with the styles repo #583

Closed maxim-belkin closed 3 years ago

maxim-belkin commented 3 years ago

This PR does two things:

  1. Merges the contents of the "Determine the proper reference to use" step into the "Sync lesson with carpentries/styles" step.
  2. Fixes the latter step so that it doesn't fail for lessons that have just synced up with the carpentries/styles.
zkamvar commented 3 years ago

This looks good. I will take a look at it a bit more closely tomorrow. Thank you for the patch!

maxim-belkin commented 3 years ago

I combined the merge --no-commit and commit steps because merge steps doesn't fail when the repo is up-to-date.

maxim-belkin commented 3 years ago

I'll update this PR after #584 is merged

maxim-belkin commented 3 years ago

This one is ready for review