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.

Sign up for GitHub

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

Jump to bottom

Introduce lint plugin #133

Merged
merged 31 commits into from
Mar 20, 2024
Merged

Introduce lint plugin #133

merged 31 commits into from
Mar 20, 2024

Conversation

tommy351
Copy link
Owner

No description provided.

@tommy351
chore(vscode): Update Jest settings
7fc56c3
@tommy351
feat(generate): Implement validation options
c6b5645
@tommy351
feat(generate): Add keepAjvErrors option
8492249
@tommy351
feat(generate): Add component to Manifest interface
2682a05
@tommy351
refactor(cli): Refactor error handling
1264450
@tommy351
feat(cli): Support validation plugins
6b6ef9a
@tommy351
feat(lint): Introduce lint plugin
cb84d78
@tommy351
feat(lint): Add no-missing-namespace rule
89ee4bb
@tommy351
chore(cli): Change issue format
7cda9b2
@tommy351
feat(lint): Implement rules
e904ce3
@tommy351
feat(require): Add resolveModule function
51e93b1
@tommy351
fix: Fix manifest type errors
c4bf57d
@tommy351
chore(build-scripts): Update rollup and swc
ec196ff
@tommy351
build: Update build scripts
b7f481b 
- File paths such as `foo/bar.ts` were written to `bar.js` previously, now they are written to `foo/bar.js` instead.
- Strip `src/` from entry paths.
- Update SWC target to es2022 because the minimum supported Node.js version is 18.
@tommy351
feat(lint): Add recommended preset
19e9f6b
@tommy351
build: Set build environment in virtual module
9bf248e
@tommy351
fix(lint): Ignore number port in valid-probe-port rule
1704b90
@tommy351
chore(lint): Update recommended preset
6b40d23
@tommy351
fix(lint): Ignore system roles in no-missing-role rule
92cb557
@tommy351
fix(lint): Allow subset of selector in valid-pod-selector rule
25ff161
@tommy351
feat(lint): Always built-in namespaces in no-missing-namespace rule
9e4948a
@tommy351
fix(lint): Allow default service account
d04775b
@tommy351
feat(lint): Ignore StatefulSet in no-missing-service rule
b079d5d
@tommy351
feat(lint): Implement string pattern
6dba546
@tommy351
feat(lint): Support gateway API in no-missing-service rule
3a94917
@tommy351
feat(lint): Add no-missing-gateway rule
49491d4
@tommy351
feat(lint): Support all kinds of gateway routes
dda0cdd
@tommy351
feat(lint): Add no-missing-gateway-class rule
fe88dcc
@tommy351 tommy351 self-assigned this Mar 19, 2024
@codecov Codecov
Copy link

codecov bot commented Mar 19, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 99.39714% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 96.94%. Comparing base (7c2e54e) to head (de88bde).

Files Patch % Lines
packages/require/src/extensions.ts 33.33% 2 Missing ⚠️
packages/cli/src/commands/generate/command.ts 50.00% 0 Missing and 1 partial ⚠️
packages/cli/src/commands/generate/env.ts 83.33% 1 Missing ⚠️
packages/cli/src/commands/migrate.ts 66.66% 0 Missing and 1 partial ⚠️
packages/generate/src/generate.ts 94.73% 1 Missing ⚠️
packages/require/src/resolve.ts 96.55% 1 Missing ⚠️
plugins/lint/src/config.ts 97.77% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##           master     #133      +/-   ##
==========================================
+ Coverage   94.57%   96.94%   +2.37%     
==========================================
  Files          74      118      +44     
  Lines        1676     2752    +1076     
  Branches      398      711     +313     
==========================================
+ Hits         1585     2668    +1083     
+ Misses         90       76      -14     
- Partials        1        8       +7

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Mar 20, 2024 •
edited
Loading

Deploying kosko with  Cloudflare Pages  Cloudflare Pages

Latest commit: de88bde
Status: ✅  Deploy successful!
Preview URL: https://db65e7f5.kosko.pages.dev
Branch Preview URL: https://lint-plugin.kosko.pages.dev

View logs

@tommy351 tommy351 merged commit ed54487 into master Mar 20, 2024
20 checks passed
@tommy351 tommy351 deleted the lint-plugin branch March 20, 2024 11:56
@github-actions github-actions bot mentioned this pull request Mar 20, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tommy351 tommy351

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tommy351