Closed tobyhodges closed 4 years ago
Thanks @tobyhodges, this looks great and tidies things up nicely. I'll give it a try on my local machine ASAP and let you know if I run into any troubles.
Note: just updated the branch to also remove .ruby-gemset
& .ruby-version
I'm going to ping @fmichonneau to respond to those questions, as I'm sure he'll do a better job than I could. François is on leave until Tuesday, though, so I'm afraid you'll likely have to wait for definitive answers.
this looks good to me! Thank you all.
Thanks François. I'll update the Makefile to match the latest version in carpentries/styles and then I think we're ready to go.
The Makefile has been synced, so I think this is ready to be merged
Thanks @fmichonneau @tobyhodges!
This should be everything you need to start using the remote theme. As well as minor changes to
.gitignore
and_config.yml
, the main differences are:_layouts
,_includes
,assets
directories) are removed - after merging, this information will be drawn directly from the remote theme repomake serve
command, to build and serve pages locally, now uses Bundler (i.e.bundle exec jekyll serve
rather thanjekyll serve
directly)I have only tested this on my local machine and GitHub - maybe you'd like to pull this branch and confirm it works on your local setup before hitting the Big Green Button?