Yatagan 2.0 release notes and migration guide#214
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #214 +/- ##
=========================================
Coverage 83.65% 83.65%
Complexity 1661 1661
=========================================
Files 200 200
Lines 7967 7967
Branches 1517 1517
=========================================
Hits 6665 6665
Misses 760 760
Partials 542 542 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| @Conditional(FeatureA::class) | ||
| @Conditional(FeatureB::class) |
There was a problem hiding this comment.
This code is technically incorrect, because these two conditionals would conflict, because they lack onlyIn selector. As in "before" block too. I understand that this is only a demo code, but I'd probably be better to demo fully correct code. We could leave only one conditional, or specify onlyIn.
|
|
||
| ### 2. Generated Component Class Names | ||
|
|
||
| Generated implementation class names have changed to be more tooling-friendly. |
There was a problem hiding this comment.
| Generated implementation class names have changed to be more tooling-friendly. | |
| Generated implementation class names have changed to be more user-friendly, so that they can be used directly. |
|
|
||
| ### 3. Subcomponent Binding Forbidden | ||
|
|
||
| Direct binding/injection of subcomponent instances is no longer allowed. Use factory methods instead. |
There was a problem hiding this comment.
I actually don't remember, but wan't this released in 1.x.y also?
There was a problem hiding this comment.
AFAIK here is a commit where the change was introduced: d6bf9d2
And as I can see, it was merged only in main with fix version 2.0.0
| ### 4. Component Factory Method Behavior | ||
|
|
||
| All methods in a component interface returning a non-root component are now treated as factory methods, regardless of parameters. |
There was a problem hiding this comment.
This section is essentially the consequence of ### 3., maybe let's merge these sections? It doesn't seem to deserve a full section on its own.
bacecek
force-pushed
the
users/bacecek/yatagan_2
branch
from
ecbaa32 to
c90b094
Compare
|
@Jeffset I guess I have fixed mentioned problems, could you please take another look? |
bacecek
force-pushed
the
users/bacecek/yatagan_2
branch
from
c90b094 to
4c22a28
Compare
|
Hi, @Jeffset I changed my mind and would like to merge the migration guide into the repository after all, and then link to it from the release tab. I also updated the README to reflect the current state of the project. Please take a look when you have time. |
2 participants
This PR introduces a markdown file for the 2.0 release notes and migration guide.
It also contains updated README.md files for new version.