Skip to content

The tsconfig include property is not supported #381

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

Closed
rabbitfang opened this issue Nov 16, 2016 · 5 comments
Closed

The tsconfig include property is not supported #381

rabbitfang opened this issue Nov 16, 2016 · 5 comments

Comments

@rabbitfang
Copy link

The include option in the tsconfig.json file is not read by grunt-ts. It basically has the same behavior as filesGlob, but is officially supported.

@evil-shrike
Copy link

+1
I've also encountered that the plugin cannot compile my project.
I have tsconfig.json with include and no exclude. I run npm run tsc to compile, where "tsc" is a npm script name:

  "scripts": {
    "tsc": "tsc",
    "test": "grunt test"
  },

It works fine.
But compiling via the plugin is failing.
Gruntfile:

    ts: {
        default : {
            tsconfig: 'true'
        }
    }

TypeScript 2.1
I'm not keed to replicate tsconfig settings in Gruntfile.

@evil-shrike
Copy link

evil-shrike commented Dec 16, 2016

And even if I specify files in Grunt task via src optons while keeping tsconfig option, the plugin ignores them and compile everything

			default : {
				tsconfig: "tsconfig.json",
				src: [
					"src/**/*.ts",
					"src/**/.*.ts",
					"../../../Client/src/**/*.ts",
					"../../../Client/src/**/.*.ts"
				]
			}

@rabbitfang
Copy link
Author

@evil-shrike As a workaround, you can update the definition to be:

{
  ts: {
    default: {
      tsconfig: {
        tsconfig: 'path/to/tsconfig.json',
        passThrough: true,
        updateFiles: false,
      }
    }
  }
}

This causes grunt-ts to call tsc with just the tsconfig.json file (and no files to compile).

@nycdotnet
Copy link
Contributor

This is implemented in 6.0.0-beta.6 on npm. Sorry for the delay. Please give it a whirl and let me know if you hit any issues.

@nycdotnet
Copy link
Contributor

I am going to close this issue as I believe it is resolved in the latest beta. Please feel free to reopen/comment. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants