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

Feature: Exclude files from 'watch' mode #2425

Closed
kchojhu opened this issue Sep 29, 2016 · 19 comments
Closed

Feature: Exclude files from 'watch' mode #2425

kchojhu opened this issue Sep 29, 2016 · 19 comments
Assignees
Labels
help wanted P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent severity2: inconvenient type: bug/fix

Comments

@kchojhu
Copy link

kchojhu commented Sep 29, 2016

I think it would be great if you can exclude files to not watch so that it doesn't recompile every time. As of now, I'm using styles.scss to override the default styles.css. Yes, I do know that component can include the scss but I rather just have 1 style file for entire application (it's very small app). Anyways, everytime I change scss files then it gets compiled twice. Thanks and definitely loving NG2.

@eight-molecules
Copy link
Contributor

Currently Webpack builds the entire project on any change, so excluding a file from being watched isn't going to do much yet. There are plans to allow partial building of projects for this sort of case, but based on responses from the Angular CLI team we should expect them after the new year begins.

@filipesilva
Copy link
Contributor

filipesilva commented Oct 8, 2016

It definitely shouldn't be compiled twice, that sounds like a bug. I tried to repro that on latest master and didn't get a double compile, so it might be fixed now.

Regarding watch exclusing... I don't think we're adding that anytime soon. The way webpack works, it watches every dependency of the app plus the stuff we tell it to watch (styles, scripts, assets). If you don't want any of those to be watched, you can just remove it from angular-cli.json.

@filipesilva filipesilva added type: bug/fix command: build needs: investigation Requires some digging to determine if action is needed labels Oct 8, 2016
@filipesilva filipesilva removed command: build needs: investigation Requires some digging to determine if action is needed type: bug/fix labels Oct 8, 2016
@awerlang
Copy link
Contributor

awerlang commented Mar 8, 2017

it watches every dependency of the app plus the stuff we tell it to watch (styles, scripts, assets)

The entire root folder is watched. Repro:

  1. Add a new folder inside app's root directory, let's call it scss;
  2. Add a new file to this folder, let's call it .gitkeep;

ng serve would trigger a recompile.

I have my scss files inside this source folder, but these are compiled externaly. So every time I save, the app is refreshed but without the latest build from sass.

I believe this is a bug, given that the scope of watched files is wider than expected.

@skulptur
Copy link

skulptur commented Mar 9, 2017

In my case it´s with pug files. I´m compiling them with gulp and expected only the generated html to trigger compilation or at least a way to exclude .pug from watch. I have them at the same folder where the html is generated.

@samartioli
Copy link

samartioli commented May 31, 2017

@spredemann Did you ever figure out a solution... I am using pug with Angular 4, and using a gulp task to compile pug to html.. problem is ng-build see's the pug changes and recompiles. Wasn't a big deal until I started using include pugfile.pug pug files...
I supposed I could ng eject and wire up webpack myself... I'd prefer not to.

@skulptur
Copy link

@samartoli no but you probably should avoid includes anyway. Write smaller components.

@edoloughlin
Copy link

Emacs creates a file called .#<filename> when you begin editing <filename>. This causes angular-cli to rebuild. I could set watchOptions.ignored to avoid this issue but this doesn't seem to be possible with angular-cli, forcing me to ng eject

@dmitrysteblyuk
Copy link

dmitrysteblyuk commented Jul 10, 2017

@filipesilva

If you don't want any of those to be watched, you can just remove it from angular-cli.json

The way I see it, everything is watched. Every time I upload a file ng rebuilds. Changing root in angular-cli.json does not help.

I guess I have to move public folder outside of root directory.

@towc
Copy link

towc commented Jul 14, 2017

yeah, how do I get it not to watch vim files? It's very very annoying

@filipesilva
Copy link
Contributor

Reopening, I can confirm that everything in src/ is indeed watched. I don't know if there's a way to configure webpack to not rebuild when random stuff is added to the folder. PR's and suggestions are welcome.

@filipesilva filipesilva reopened this Jul 17, 2017
@filipesilva filipesilva self-assigned this Jul 17, 2017
@filipesilva filipesilva added help wanted P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent severity2: inconvenient type: bug/fix labels Jul 17, 2017
@dmitrysteblyuk
Copy link

I believe everything in project directory is watched (where angular-cli.json is located) not only in src/. I have app root set to "src" in angular-cli.json, but still ng recompiles when any of my project files is saved.

@simpers
Copy link

simpers commented Jul 19, 2017

Is this being worked on? I am just like @edoloughlin using Emacs and right now I get this every time I edit a scss file and saving it before it finishes compiling:

webpack: Compiling... 94% asset optimization(node:6150) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: ENOENT: no such file or directory, stat '/home/simon/arbete/gor-cli/src/assets/scss/.#index.scss' (node:6150) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

@jaimealsilva
Copy link

Same issue as #4593

@redbmk
Copy link

redbmk commented Aug 17, 2017

I'm looking into a fix for this. I'm thinking it would be an option passed in, similar to poll (maybe called ignored) that lets you set an anymatch string via the command line, or an anymatch string/array of strings via an option in .angular-cli.json.

If I ng eject and add ignored: "**/*.swp" to devServer.watchOptions it will ignore writes to any *.swp file. However, if I add ignored: "**/*.swp" to watchOptions in the dev server config in angular-cli (after modifying the spec so it accepts the option) it still writes on those files.

Any ideas what could be going wrong here?

@redbmk
Copy link

redbmk commented Aug 17, 2017

I take some of it back...it is working when I do the steps recommended in npm link and run this modified code in another project, but the test I have written is not passing. Something about this triggers a rebuild when it shouldn't. Can anybody see what's wrong with this test? When I run the linked version I'm invoking it the same way (ng serve --ignored=**/*.swp). I also have a console.log showing me the configuration and it shows the same in the test as it does when using it in practice.

import {
  killAllProcesses,
  waitForAnyProcessOutputToMatch,
  execAndWaitForOutputToMatch
} from '../../utils/process';
import {writeFile, appendToFile} from '../../utils/fs';
import {expectToFail} from '../../utils/utils';

const webpackGoodRegEx = /webpack: Compiled successfully./;

export default function() {         
  const ignoreCmd = `--ignored=**/*.swp`;
  
  return execAndWaitForOutputToMatch('ng', ['serve', ignoreCmd], webpackGoodRegEx)
    .then(() => appendToFile('src/main.ts', 'console.log(1);')) 
    .then(() => waitForAnyProcessOutputToMatch(webpackGoodRegEx, 10000))
    
    .then(() => writeFile('src/.main.ts.swp', 'console.log(1);')) //FIXME: this still triggers a rebuild
    .then(() => expectToFail(() => waitForAnyProcessOutputToMatch(webpackGoodRegEx, 10000)))
    
    .then(() => appendToFile('src/main.ts', 'console.log(1);'))
    .then(() => waitForAnyProcessOutputToMatch(webpackGoodRegEx, 10000))
    .then(() => killAllProcesses(), (err: any) => {
      killAllProcesses();
      throw err;
    });
}   

@filipesilva
Copy link
Contributor

Closed via #7310, src/ is no longer watched unconditionally.

@jbojcic1
Copy link

jbojcic1 commented Mar 5, 2019

For library generated with cli it seems to be watching entire library folder. @filipesilva I assume that currently it's not possible to add exception for library?

@Robert-Baindourov
Copy link

Robert-Baindourov commented Jul 10, 2019

I don't have this anular-cli.json file that has been mentioned. Using angular 7.
Is it serve.json now?
Can You link to an example of how to setup exclusion?

robert@pop-os:~/bose_project/Raphael-App-Framework$ sudo find / -iname angular-cli.json
find: ‘/run/user/1000/gvfs’: Permission denied
robert@pop-os:~/bose_project/Raphael-App-Framework$ sudo find / -iname serve.json
find: ‘/run/user/1000/gvfs’: Permission denied
/home/robert/bose_project/Raphael-App-Framework/node_modules/@angular/cli/commands/serve.json
/home/robert/bose_project/angular-electron/node_modules/@angular/cli/commands/serve.json
/home/robert/.nvm/versions/node/v12.4.0/lib/node_modules/@angular/cli/commands/serve.json
/home/robert/Raphael-App-Framework/node_modules/@angular/cli/commands/serve.json
/home/robert/.cache/yarn/v4/npm-@angular-cli-7.3.3-b357000385aa6c75b001cb9fa7982ef3ce02c423/node_modules/@angular/cli/commands/serve.json
/var/backups/backintime/pop-os/robert/1/new_snapshot/backup/home/robert/.nvm/versions/node/v12.4.0/lib/node_modules/@angular/cli/commands/serve.json

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
help wanted P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent severity2: inconvenient type: bug/fix
Projects
None yet
Development

Successfully merging a pull request may close this issue.