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

Add support for "exclude" in tsconfig.json #3043

Closed
mhegazy opened this issue May 5, 2015 · 1 comment
Closed

Add support for "exclude" in tsconfig.json #3043

mhegazy opened this issue May 5, 2015 · 1 comment
Assignees
Labels
Bug A bug in TypeScript Fixed A PR has been merged for this issue

Comments

@mhegazy
Copy link
Contributor

mhegazy commented May 5, 2015

tsconfig.json picks all files in the containing folder by default, this is problematic with "node_modules" that contain .ts/.d.ts that are not relevant to the project. "files" is not useful here because of the lack of support for globs (see #1927).

Proposal:

  • support a new property "exclude" that specifies a list of directories/files to exclude.
  • "exclude" is ignored if "files" is present.
{
    "compilerOptions": {
        "out": "../../built/local/tsc.js",
        "sourceMap": true
    },
    "exclude": [
        "script/helper.ts",
        "node_modules"
    ]
}

Note: This is needed if we want any of gulp/grunt/task runner/ProjectK support in VS .

@mhegazy mhegazy added the Bug A bug in TypeScript label May 5, 2015
@mhegazy mhegazy modified the milestones: TypeScript 1.5.2, TypeScript 1.6 May 5, 2015
@mhegazy mhegazy self-assigned this May 5, 2015
@ahejlsberg ahejlsberg assigned ahejlsberg and unassigned mhegazy May 16, 2015
@ahejlsberg ahejlsberg changed the title Add support "excludes" in tsconfig.json Add support for "exclude" in tsconfig.json May 16, 2015
@briandipalma
Copy link

Would glob support be enough not to require this feature? I've also had this issue where node_modules was being built but with glob support I'd be fine.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A bug in TypeScript Fixed A PR has been merged for this issue
Projects
None yet
Development

No branches or pull requests

3 participants