Merge pull request #641 from MadeByMike/add-custom-watcher

Add addWatchTarget() option to config

.github/ISSUE_TEMPLATE/i-have-a-question-about-eleventy.md

@@ -1,10 +1,7 @@
 ---
 name: I have a question about Eleventy
 about: e.g. “How do I do this in Eleventy?” or “Can Eleventy do this?”
-title: ''
+title: ""
 labels: education
-assignees: ''
-
+assignees: ""
 ---
-
-

.github/ISSUE_TEMPLATE/i-m-having-a-problem-with-eleventy.md

@@ -1,17 +1,17 @@
 ---
 name: I’m having a problem with Eleventy
 about: Create a report to help us improve
-title: ''
+title: ""
 labels: needs-triage
-assignees: ''
-
+assignees: ""
 ---
 
 **Describe the bug**
 A clear and concise description of what the bug is.
 
 **To Reproduce**
 Steps to reproduce the behavior:
+
 1. Go to '...'
 2. Click on '....'
 3. Scroll down to '....'
@@ -24,8 +24,9 @@ A clear and concise description of what you expected to happen.
 If applicable, add screenshots to help explain your problem.
 
 **Environment:**
- - OS and Version: [e.g. Windows/Mac/Linux]
- - Eleventy Version [via `eleventy --version` or `npx @11ty/eleventy --version`]
+
+- OS and Version: [e.g. Windows/Mac/Linux]
+- Eleventy Version [via `eleventy --version` or `npx @11ty/eleventy --version`]
 
 **Additional context**
 Add any other context about the problem here.

.github/ISSUE_TEMPLATE/i-want-eleventy-do-to-this-new-thing.md

@@ -1,10 +1,9 @@
 ---
 name: I want Eleventy do to this new thing
 about: Suggest an idea for this project
-title: ''
+title: ""
 labels: enhancement
-assignees: ''
-
+assignees: ""
 ---
 
 **Is your feature request related to a problem? Please describe.**

.travis.yml

@@ -13,4 +13,6 @@ sudo: false
 os:
   - linux
   - osx
-  - windows
+  - windows
+env:
+  - YARN_GPG=no

src/Eleventy.js

@@ -73,8 +73,7 @@ class Eleventy {
 
     /** @member {Object} - tbd. */
     this.watchTargets = new EleventyWatchTargets();
-
-    /** @member {Object} - tbd. */
+    this.watchTargets.add(this.config.additionalWatchTargets);
     this.watchTargets.watchJavaScriptDependencies = this.config.watchJavaScriptDependencies;
   }
 

src/UserConfig.js

@@ -55,6 +55,7 @@ class UserConfig {
     // this.userExtensionMap = {};
     // this.templateExtensionAliases = {};
     this.watchJavaScriptDependencies = true;
+    this.additionalWatchTargets = [];
     this.browserSyncConfig = {};
 
     // using Map to preserve insertion order
@@ -560,6 +561,10 @@ class UserConfig {
   //   this.templateExtensionAliases[extension] = targetKey;
   // }
 
+  addWatchTarget(additionalWatchTargets) {
+    this.additionalWatchTargets.push(additionalWatchTargets);
+  }
+
   setWatchJavaScriptDependencies(watchEnabled) {
     this.watchJavaScriptDependencies = !!watchEnabled;
   }
@@ -605,6 +610,7 @@ class UserConfig {
       experiments: this.experiments,
       // templateExtensionAliases: this.templateExtensionAliases,
       watchJavaScriptDependencies: this.watchJavaScriptDependencies,
+      additionalWatchTargets: this.additionalWatchTargets,
       browserSyncConfig: this.browserSyncConfig,
       frontMatterParsingOptions: this.frontMatterParsingOptions,
       dataExtensions: this.dataExtensions

test/TemplateConfigTest.js

@@ -328,3 +328,15 @@ test("Properly throws error when config returns a Promise", t => {
     );
   });
 });
+
+test(".addWatchTarget adds a watch target", t => {
+  eleventyConfig.reset();
+  eleventyConfig.addWatchTarget("/testdirectory/");
+
+  let templateCfg = new TemplateConfig(
+    require("../config.js"),
+    "./test/stubs/config.js"
+  );
+  let cfg = templateCfg.getConfig();
+  t.deepEqual(cfg.additionalWatchTargets, ["/testdirectory/"]);
+});