addWatchTarget ignored when used in a custom config file

[gael-ian]

Hi,

When used in a custom config file, addWatchTarget is ignored.

To Reproduce

Steps to reproduce the behavior:

1. Clone this repository and install dependencies.
   It's nothing more than Eleventy's Getting started with 2 config files: .eleventy.js and config/eleventy.js. Both contain a call to addWatchTarget.
2. Run DEBUG=Eleventy npx @11ty/eleventy --serve.
3. addWatchTarget works:

   Eleventy Watching for changes to: [ './assets/**', './**/*.liquid', […], './**/*.11tydata.js' ] +5ms
   

4. Rename the default .eleventy.js: mv .eleventy.js ._eleventy.js
5. Run DEBUG=Eleventy npx @11ty/eleventy --config=config/eleventy.js --serve
6. addWatchTarget is broken:

   Eleventy Watching for changes to: [ './**/*.liquid', […], './**/*.11tydata.js' ] +5ms
   

Expected behavior

addWatchTarget should work when called from a custom config file.

As a bonus, should .eleventy.js even be loaded when Eleventy is called with a custom config file ?

[binyamin]

Confirmed. addWatchTarget() does not work with any custom config file.
@gael-ian Thanks for the detailed bug report.

[daflh]

Has this been fixed?

[TigersWay]

Still not fixed and pretty annoying! Any trouble with #1568 from @dafiulh ?

[davidbwaters]

OMG, thanks for this. Needs fix or note.

[einomi]

Any updates on this? Looks like this PR adds a fix for this issue?

[zachleat]

As a bonus, should .eleventy.js even be loaded when Eleventy is called with a custom config file ?

Confirmed this as already fixed (tested in 2.x and 1.x)

As for the PR #1568 I was able to confirm that this issue is no longer relevant on 1.x:

DEBUG=Eleventy* npx @11ty/eleventy --watch --config=eleventy.config.js

[11ty] Wrote 0 files in 0.08 seconds (v1.0.1)
Eleventy Watching for changes to: [ './assets/**',          './**/*.liquid', './**/*.ejs',           './**/*.md', './**/*.hbs',           './**/*.mustache', './**/*.haml',          './**/*.pug', './**/*.njk',           './**/*.html', './**/*.11ty.js',       './**/*.11ty.cjs', './_includes/**',       './_data/**', './eleventy.config.js', './**/*.json', './**/*.11tydata.cjs',  './**/*.11tydata.js' ] +5ms

// eleventy.config.js
module.exports = function(eleventyConfig) {
  eleventyConfig.addWatchTarget("assets/**");
};

Thanks everyone! (please speak up if I’m wrong!)