-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[Cleanup] Remove deprecated APIs #529
[Cleanup] Remove deprecated APIs #529
Conversation
That includes following properties that are declared on ASDisplayNode and ASLayoutSpec: spacingBefore, spacingAfter, flexGrow, flexShrink, flexBasis, alignSelf, ascender, descender, sizeRange and layoutPosition.
-visibilityDidChange: -visibleStateDidChange: -displayStateDidChange: -loadStateDidChange:
…thod in ASPagerDelegate
🚫 CI failed with log |
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.
Awesome, agree with @nguyenhuy, feels good to see that code going away. We should do a bigger release, 2.5 sounds fine for me.
@nguyenhuy We should also run it against P to see if we run into any issues we can prevent before merging / releasing a new version.
@maicki Yes, I have a pending branch that fixes issues in P. Gonna leave this PR for another day or two before merging, in case someone else wants to chime in. |
Generated by 🚫 Danger |
* Remove preferredFrameSize * Remove -measure: * Remove -measureWithSizeRange: * Remove ASLayoutable * Remove .name * Remove deprecated style forwardings That includes following properties that are declared on ASDisplayNode and ASLayoutSpec: spacingBefore, spacingAfter, flexGrow, flexShrink, flexBasis, alignSelf, ascender, descender, sizeRange and layoutPosition. * Remove usesImplicitHierarchyManagement * Remove deprecated range update callbacks: -visibilityDidChange: -visibleStateDidChange: -displayStateDidChange: -loadStateDidChange: * Remove -clearFetchedData * Remove -cancelLayoutTransitionsInProgress * Remve ASDisplayNode+Deprecated.h * Remove ASLayoutRangeTypeRender and ASLayoutRangeTypeFetchData * Remove -[ASTableView clearContents] * Remove reloadDataImmediately * Remove ASStaticLayoutSpec * Remove ASDimensionDeprecated * Remove optional -pagerNode:constrainedSizeForNodeAtIndex: delegate method in ASPagerDelegate * Remove suppressesInvalidCollectionUpdateExceptions * Remove -[ASCollectionViewLayoutInspector initWithCollectionView] * Remove ASVideoPlayerNode.loadAssetWhenNodeBecomesVisible * Update CHANGELOG * Update license of ASLayoutSpecTests.m * Update examples/PagerNode * Remove ASEnvironmentTraitCollection * Remove -ASViewController.nodeConstrainedSize * More on removing ASLayoutable
Remove APIs that have been deprecated since 2.0 and/or for at least 6 months:
-ASDisplayNode.preferredFrameSize
.-[ASDisplayNode measure:]
and-[ASDisplayNode measureWithSizeRange:]
.ASLayoutable
.-ASDisplayNode.name
.spacingBefore
,spacingAfter
,flexGrow
,flexShrink
,flexBasis
,alignSelf
,ascender
,descender
,sizeRange
andlayoutPosition
ofASDisplayNode
andASLayoutSpec
.+ASDisplayNode.usesImplicitHierarchyManagement
.ASDisplayNode
:-visibilityDidChange:
,-visibleStateDidChange:
,-displayStateDidChange:
,-loadStateDidChange:
.-clearFetchedData
ofASDisplayNode
andASTableView
.-[ASDisplayNode cancelLayoutTransitionsInProgress]
.ASLayoutRangeTypeRender
andASLayoutRangeTypeFetchData
.-[ASTableView clearContents]
.-reloadDataImmediately
of table and collection views and nodes.ASStaticLayoutSpec
andASDimensionDeprecated
.-pagerNode:constrainedSizeForNodeAtIndex:
delegate method ofASPagerDelegate
.+ASDisplayNode.suppressesInvalidCollectionUpdateExceptions
.-[ASCollectionViewLayoutInspector initWithCollectionView:]
.-ASVideoPlayerNode.loadAssetWhenNodeBecomesVisible
.ASEnvironmentTraitCollection
.The next release should be a major release (2.5). Sorry for the huge PR, but I couldn't stop digging down the rabbit hole. It feels so good!