-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
refactor(module:*): remove method parameter decorators #8567
base: master
Are you sure you want to change the base?
refactor(module:*): remove method parameter decorators #8567
Conversation
This preview will be available after the AzureCI is passed. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #8567 +/- ##
==========================================
- Coverage 91.51% 91.50% -0.01%
==========================================
Files 534 534
Lines 18425 18404 -21
Branches 2905 2904 -1
==========================================
- Hits 16861 16841 -20
+ Misses 1244 1242 -2
- Partials 320 321 +1 ☔ View full report in Codecov by Sentry. |
911de20
to
fd95dc9
Compare
@HyperLife1119 review in progress :) |
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
@HyperLife1119 please resolve the conflitcts then it willbe good |
fd95dc9
to
21e8965
Compare
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
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: N/A
What is the new behavior?
逐步迁移到标准装饰器,作为第一步,移除所有方法参数装饰器。
Does this PR introduce a breaking change?
Other information