Skip to content
This repository was archived by the owner on Sep 25, 2019. It is now read-only.

feat(scripts): stop caching challenges and improve build scripts #179

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

ojeytonwilliams
Copy link
Contributor

Avoid caching challenges so that changes can be hotloaded into Gatsby. Add scripts to watch
challenges and atomically copy them to the dist directory, without minification.

ISSUES CLOSED: #68

Description

Unfortunately linting changed a fair bit of the original. My main changes are to add some build scripts, some gulp tasks and use fs to read the challenge json, rather than require

Pre-Submission Checklist

  • Your pull request targets the dev branch.
  • Branch starts with either fix/, feature/, or translate/ (e.g. fix/challenge-tests)
  • All new and existing tests pass the command npm test.
  • Use npm run commit to generate a conventional commit message.
    Learn more here: https://conventionalcommits.org/#why-use-conventional-commits
  • The changes were done locally on your machine and NOT GitHub web interface.
    If they were done on the web interface you have ensured that you are creating conventional commit messages.

Checklist:

  • Tested changes locally.
  • Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #68

@raisedadead raisedadead added the status: ready for QA To be applied to PR that are ready for QA. label Jul 27, 2018
@raisedadead
Copy link
Member

raisedadead commented Jul 30, 2018

Hi @ojeytonwilliams can we get this rebased against the latest dev branch?

git pull --rebase upstream dev

@ojeytonwilliams ojeytonwilliams force-pushed the fix/challenge-caching branch from 8af8061 to 058a9d6 Compare July 30, 2018 12:15
@ojeytonwilliams
Copy link
Contributor Author

Apparently I'm an idiot and didn't save... let me see if I can fix this.

Avoid caching challenges so that changes can be hotloaded into Gatsby.  Add scripts to watch
challenges and atomically copy them to the dist directory, without minification.

ISSUES CLOSED: freeCodeCamp#68
@ojeytonwilliams ojeytonwilliams force-pushed the fix/challenge-caching branch from 058a9d6 to ffe7634 Compare July 30, 2018 12:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: ready for QA To be applied to PR that are ready for QA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Synchronizing learn with curriculum during development
2 participants