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: add more well-known fields #4

Merged
merged 6 commits into from
Aug 25, 2022
Merged

feat: add more well-known fields #4

merged 6 commits into from
Aug 25, 2022

Conversation

JounQin
Copy link
Contributor

@JounQin JounQin commented Aug 24, 2022

close #1

@JounQin JounQin changed the title Add more default fields docs: add more well-known fields Aug 24, 2022
@JounQin JounQin changed the title docs: add more well-known fields feat: add more well-known fields Aug 24, 2022
@JounQin
Copy link
Contributor Author

JounQin commented Aug 24, 2022

/home/runner/work/clean-pkg-json/clean-pkg-json/src/default-keep-properties.ts
Error:   25:1  error  Trailing spaces not allowed  no-trailing-spaces
Error:   59:1  error  Trailing spaces not allowed  no-trailing-spaces

/home/runner/work/clean-pkg-json/clean-pkg-json/src/index.ts
Warning:   33:23  warning  Unexpected any. Specify a different type                            @typescript-eslint/no-explicit-any
Warning:   35:3   warning  Unexpected console statement                                        no-console
Warning:   39:2   warning  Async arrow function has a complexity of 11. Maximum allowed is [10](https://github.com/privatenumber/clean-pkg-json/runs/8005753620?check_suite_focus=true#step:6:11)  complexity
Warning:   84:3   warning  Unexpected console statement                                        no-console

I have no idea why CI fails.

src/default-keep-properties.ts Outdated Show resolved Hide resolved
src/default-keep-properties.ts Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
src/default-keep-properties.ts Outdated Show resolved Hide resolved
src/default-keep-properties.ts Outdated Show resolved Hide resolved
@privatenumber
Copy link
Owner

Linting fails because you have a indentation on a line with no text. You can run pnpm lint --fix to fix.

JounQin and others added 2 commits August 25, 2022 08:26
Co-authored-by: hiroki osame <[email protected]>
@JounQin JounQin requested a review from privatenumber August 25, 2022 02:12
@JounQin
Copy link
Contributor Author

JounQin commented Aug 25, 2022

Linting fails because you have a indentation on a line with no text. You can run pnpm lint --fix to fix.

😅 lint-staged and simple-git-hooks/husky should be configured.

@privatenumber
Copy link
Owner

Awesome work, thanks @JounQin

@privatenumber privatenumber merged commit d43d862 into privatenumber:develop Aug 25, 2022
@privatenumber
Copy link
Owner

🎉 This PR is included in version 1.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@JounQin JounQin deleted the patch-1 branch August 25, 2022 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: add more well-known fields by default
2 participants