Skip to content

ci: add code style hook checksum - #301

Merged
MadCcc merged 9 commits into
react-component:masterfrom
Wxh16144-forks:wuxh/lint-config
Jan 16, 2023
Merged

ci: add code style hook checksum#301
MadCcc merged 9 commits into
react-component:masterfrom
Wxh16144-forks:wuxh/lint-config

Conversation

@Wxh16144

Copy link
Copy Markdown
Contributor

No description provided.

@vercel

vercelBot commented Jan 13, 2023

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

NameStatusPreviewUpdated
collapse✅ Ready (Inspect)Visit PreviewJan 13, 2023 at 10:02AM (UTC)

@codecov

codecovBot commented Jan 13, 2023

Copy link
Copy Markdown

Codecov Report

Merging #301 (b621614) into master (ca4067a) will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@## master #301 +/- ##
=======================================
Coverage 98.85% 98.85% =======================================
Files 4 4 Lines 87 87 Branches 33 33 =======================================
Hits 86 86 Misses 1 1 
Impacted FilesCoverage Δ
src/Collapse.tsx100.00% <ø> (ø)
src/Panel.tsx100.00% <ø> (ø)
src/PanelContent.tsx90.90% <ø> (ø)
src/index.tsx100.00% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Comment threadpackage.json Outdated
"lint-staged": {
"*.{ts,tsx,js,jsx}": [
"npm run prettier -- --ignore-unknown",
"npm run lint -- --fix"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

https://github.com/react-component/input/blob/master/package.json#L87
这样简单点,eslint 在 lint 时候跑就好了

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

我去检查看看,如果 prettier 和 eslint 没有冲突的话应该可行的。

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

git add 已经不需要了,https://github.com/okonet/lint-staged#v10

@MadCcc
MadCcc merged commit 53b6294 into react-component:masterJan 16, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@Wxh16144@MadCcc