Skip to content
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

chore: refactor to a single watcher #1133

Merged
merged 2 commits into from Sep 2, 2020
Merged

chore: refactor to a single watcher #1133

merged 2 commits into from Sep 2, 2020

Conversation

olizilla
Copy link
Collaborator

@olizilla olizilla commented Sep 1, 2020

  • add watch.js to configure chokidar.
  • pass watcher to each tool via an exported configureWatcher method.

If run directly, the tool assumes it should process all things it cares about. If required as module, it does nothing, and allows the calling code to use it to configure the watcher.

License: MIT
Signed-off-by: Oli Evans oli@tableflip.io

- add watch.js to configure chokidar.
- pass watcher to each tool via an exported `configureWatcher` method.
- allow tools to be run as scripts or required as modules

License: MIT
Signed-off-by: Oli Evans <oli@tableflip.io>
@olizilla
Copy link
Collaborator Author

olizilla commented Sep 1, 2020

This is based off of the tic-toc branch for #1122 as re-working the watcher is a significant change that is unrelated to the goal of that PR.

License: MIT
Signed-off-by: Oli Evans <oli@tableflip.io>
@hugomrdias hugomrdias added this to In progress in Sprint 31 Aug 2020 via automation Sep 2, 2020
@olizilla olizilla merged commit 28fcc0d into tic-toc Sep 2, 2020
Sprint 31 Aug 2020 automation moved this from In progress to Done Sep 2, 2020
@olizilla olizilla deleted the one-watcher branch September 2, 2020 14:25
olizilla added a commit that referenced this pull request Sep 2, 2020
- Avoid building the ToC and the dashboard on the client side by parse data from dom and building the toc from data. Generates a toc.json file in the hugo data dir, which is then used by the toc partial and the dashboard-spec shortcode. 

fixes #1125 

- Refactor watchers. Add watch.js to configure chokidar. Pass watcher to each tool via an exported `configureWatcher` method. Allow tools to be run as scripts or required as modules

see #1133

License: MIT
Signed-off-by: Oli Evans <oli@tableflip.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants