Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

Changes to the reveal.js dependency #42

Closed
hectorsector opened this issue Mar 25, 2020 · 1 comment
Closed

Changes to the reveal.js dependency #42

hectorsector opened this issue Mar 25, 2020 · 1 comment

Comments

@hectorsector
Copy link
Member

The template repo relies on reveal.js. At the time we created this course, the only way to utilize reveal.js was through a submodule.

As of earlier today, the submodule was checked out to a 2017 commit, so I did I tried to make this a little easier today.

Today, here are the changes implemented:

  • We now consume reveal.js via its Node module
  • We no longer use the reveal.js submodule
  • Because we rely on GitHub Pages for static hosting, we reference files in the node_modules folder directly

I'll close this out in a couple of days but thought it was worth explaining for folks that might come in behind me trying to make changes to this course.

@lmkeston
Copy link
Contributor

Seems to be working properly!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants