-
-
Notifications
You must be signed in to change notification settings - Fork 757
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
Prepare RC13 #1462
Prepare RC13 #1462
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1462 +/- ##
=========================================
Coverage 74.68% 74.68%
Complexity 1720 1720
=========================================
Files 327 327
Lines 5238 5238
Branches 950 950
=========================================
Hits 3912 3912
Misses 805 805
Partials 521 521 Continue to review full report at Codecov.
|
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.
Are we really ready for this? Frankly speaking, I don't think so.
Sure, we have more than I ever wanted to implement for a 1.0. Idk what should be missing? I want to start making minor releases ^^. |
There have been so many API changes in the last few releases. (Including from RC12 to 1.0.0) What's the benefit for having a 1.0.0 instead of a RC13? Personally, I don't see one. |
Actually your right, things like LargeClass and LongMethod rework need a RC13 because they are active by default. Will change to RC13. |
13e060b
to
4520344
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.
Thanks 🙏
No description provided.