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

not an issue: tsconfig.json doesn't match npm package installed #737

Closed
sqlsolver opened this issue Jul 14, 2016 · 7 comments
Closed

not an issue: tsconfig.json doesn't match npm package installed #737

sqlsolver opened this issue Jul 14, 2016 · 7 comments

Comments

@sqlsolver
Copy link

The /node_modules/ng2-bootstrap/tsconfig.json files section specifies the following:
"files": [
"./typings/index.d.ts",
"./demo/custom-typings.d.ts",
"./ng2-bootstrap.ts"
]
however in VS2015 the build fails since expected folders are missing, in fact the entire demo and typings directories are missing from the version you get when running 'npm install ng2-bootstrap' and trying 'npm install https://github.com/valor-software/ng2-bootstrap.git' results in an installation with nothing but .md files - no javascript in the component folders (and also no demo folder). The build succeeds once the files entry is removed.

@valorkin
Copy link
Member

why do you care about tsconfig from npm?
ps: just did fresh install of ng2-bootstrap
all in place

@valorkin valorkin changed the title tsconfig.json doesn't match npm package installed no an issue: tsconfig.json doesn't match npm package installed Jul 14, 2016
@valorkin valorkin changed the title no an issue: tsconfig.json doesn't match npm package installed not an issue: tsconfig.json doesn't match npm package installed Jul 14, 2016
@sqlsolver
Copy link
Author

why do you care about tsconfig from npm?
Because the tsconfig referencing non-existent directories causes my build to fail.

ps: just did fresh install of ng2-bootstrap
all in place
Do you mean you added the missing demo and typings directories to your distribution?

@valorkin
Copy link
Member

If you want demo page you need to clone repository.

If you want to use ng2-bootstrap you don't need it

@rrfenton
Copy link

rrfenton commented Aug 29, 2016

Why is this closed? This is an actual issue.

You shouldn't be including items from your build process in the NPM module, especially if it breaks other peoples builds.

I'm encountering the same issue. The way Visual Studio handles typescript is it parses through it's solution folder for ANY tsconfig (even folders excluded in your root level tsconfig). Since the package includes a tsconfig.json, with folder and file references that are NOT in the release, the packages is causing build failures for Visual Studio users.

Seeing as how these files have no use for people using the NPM package (they should only care about the *.js and *.d.ts files), why are they included at all? There are a bunch of files that are of no help to the user at the root level of the package (tsconfig, protractor.conf, gulpfile.js, just a bunch of build stuff).

Since these files are causing build errors for some users, and are completely unneeded for the package, they should be removed from the NPM package altogether.

@dguisinger
Copy link

+1 Reopen this, it is an issue with the latest build still and Visual Studio
I just wasted an hour trying to figure out what the cause of my error messages was.

@Martin-Luft
Copy link
Contributor

I have no problems with VS Code...

@Martin-Luft Martin-Luft reopened this Sep 20, 2016
@Martin-Luft
Copy link
Contributor

@valorkin should we exclude the tsconfig.json from the npm package? Would it do any harm?

jtomaszewski pushed a commit to jtomaszewski/ng2-bootstrap that referenced this issue Oct 22, 2016
Gxerxes pushed a commit to Gxerxes/ng2-bootstrap that referenced this issue Oct 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants