-
Notifications
You must be signed in to change notification settings - Fork 363
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
chore: enable importas for lint #3603
Conversation
Signed-off-by: shawnh2 <[email protected]>
Signed-off-by: shawnh2 <[email protected]>
Signed-off-by: shawnh2 <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3603 +/- ##
==========================================
+ Coverage 68.17% 68.20% +0.03%
==========================================
Files 168 168
Lines 20484 20484
==========================================
+ Hits 13964 13971 +7
+ Misses 5511 5507 -4
+ Partials 1009 1006 -3 ☔ View full report in Codecov by Sentry. |
alias: apiextensionsv1 | ||
- pkg: sigs.k8s.io/mcs-api/pkg/apis/v1alpha1 | ||
alias: mcsapiv1a1 | ||
- pkg: k8s.io/api/(\w+)/(v[\w\d]+) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's this extra one
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
match in regexp, like matching k8s.io/api/policy/v1
, the alias will be policyv1
. same for k8s.io/api/core/v1
etc
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah cool
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM thanks !
What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #