Skip to content

Commit

Permalink
feat: add .idea to .npmignore for webstorm developers (projen#236)
Browse files Browse the repository at this point in the history
Closes projen#222 

*Description of changes:*

Saw a typo on the readme, decided to also fix an easy issue.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
martzmakes authored Oct 27, 2020

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
1 parent 2b65cd9 commit 4e6382a
Showing 3 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions .npmignore
Original file line number Diff line number Diff line change
@@ -7,6 +7,7 @@ dist
/tsconfig.json
/.github
/.vscode
/.idea
/.projenrc.js
/tsconfig.jest.json
/coverage
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -148,7 +148,7 @@ Some examples for features built-in to project types:
* Automated releases to npm, maven, NuGet and PyPI
* Mergify configuration
* LICENSE file generation
* gitignore + npmigonre management
* gitignore + npmignore management
* Node "engines" support with coupling to CI build environment and @types/node
* Anti-tamper: CI builds will fail if a synthesized file is modified manually

1 change: 1 addition & 0 deletions src/typescript.ts
Original file line number Diff line number Diff line change
@@ -219,6 +219,7 @@ export class TypeScriptProject extends NodeProject {
this.npmignore?.exclude('/tsconfig.json');
this.npmignore?.exclude('/.github');
this.npmignore?.exclude('/.vscode');
this.npmignore?.exclude('/.idea');
this.npmignore?.exclude('/.projenrc.js');

// the tsconfig file to use for estlint (if jest is enabled, we use the jest one, otherwise we use the normal one).

0 comments on commit 4e6382a

Please sign in to comment.