Skip to content
This repository has been archived by the owner on Jul 2, 2019. It is now read-only.

Drop gulp #9

Merged
merged 4 commits into from
Feb 23, 2018
Merged

Drop gulp #9

merged 4 commits into from
Feb 23, 2018

Conversation

kamakiri01
Copy link
Contributor

このpull requestが解決する内容

開発環境におけるgulp利用をやめます。

概要

  • gulp 削除
  • gulpタスクをnpm scriptに移行
  • いくつかのパッケージ削除

破壊的な変更を含んでいるか?

  • なし

tslint.json Outdated
@@ -16,12 +16,17 @@
"interface-name": false,
"jsdoc-format": true,
"label-position": true,
"label-undefined": true,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TSLint 4.0で廃止? palantir/tslint#1507
コンパイラによりカバーされます
palantir/tslint#877

tslint.json Outdated
"no-debugger": true,
"no-duplicate-key": true,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TSLint 4.0で廃止
コンパイラによりカバーされます
palantir/tslint#885

tslint.json Outdated
@@ -48,7 +52,6 @@
"no-trailing-whitespace": true,
"no-unused-expression": true,
"no-unused-variable": true,
"no-unreachable": true,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TSLint 4.0で廃止
コンパイラによりカバーされます
palantir/tslint#661

package.json Outdated
"prepublish": "gulp clean && gulp compile",
"clean": "gulp clean",
"prepublish": "npm run clean && npm run build",
"clean": "rimraf lib && rimraf spec/build",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ディレクトリ削除であることをわかりやすくするために rimraf ./lib && rimraf ./spec/build としてしまっても良さそうに思いました。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

荘ですね、直します

@yu-ogi yu-ogi self-requested a review February 22, 2018 07:48
@kamakiri01
Copy link
Contributor Author

publish される成果物に変更はないので、 version はbump されません。

@kamakiri01 kamakiri01 merged commit c24179e into master Feb 23, 2018
@kamakiri01 kamakiri01 deleted the drop-gulp branch February 23, 2018 10:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants