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

feat: support pre-commit config #21

Merged
merged 5 commits into from Jun 5, 2019
Merged

feat: support pre-commit config #21

merged 5 commits into from Jun 5, 2019

Conversation

zombieJ
Copy link
Contributor

@zombieJ zombieJ commented Jun 5, 2019

Add pre commit script support:

export default {
  cjs: 'babel',
  esm: { type: 'babel', importLibToEs: true },
+  preCommit: {
+    eslint: true,
+    prettier: true,
+  },
};
$ git add .
$ git commit -m "remove this"
✔  success   prettier success!

/Users/jilin/projects/antd/rc-select/src/index.ts
  6:7  warning  'a' is assigned a value but never used  @typescript-eslint/no-unused-vars

✖ 1 problem (0 errors, 1 warning)

✔  success   eslint success!
[tmp_pr f2ea3a3] remove this
 1 file changed, 2 insertions(+)

cc @chenshuai2144 @afc163 @yesmeck

@zombieJ zombieJ requested a review from sorrycc June 5, 2019 07:58
@sorrycc sorrycc changed the title feat: Support pre-commit config feat: support pre-commit config Jun 5, 2019
@sorrycc sorrycc merged commit 2853c39 into master Jun 5, 2019
@delete-merged-branch delete-merged-branch bot deleted the pre-commit branch June 5, 2019 08:13
PeachScript pushed a commit that referenced this pull request Aug 26, 2022
* feat: convert alias from tsconfig paths

* chore: support winPath

* chore: support winPath

* chore: support winPath

* chore: add debug log

* chore: add debug console

* chore: add debug console

* chore: add debug console

* chore: support winPath

* chore: support winPath

* chore: code style

* chore: remove console

* chore: code style

Co-authored-by: xierenhong <xierenhong@kuaishou.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants