Skip to content
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

deprecate ASGraphicsCreateImageWithTraitCollectionAndOptions #1824

Conversation

vovasty
Copy link
Contributor

@vovasty vovasty commented May 20, 2020

This is a followup to #1814

Summary of changes:

  • deprecate ASGraphicsCreateImageWithTraitCollectionAndOptions in favor of ASGraphicsCreateImage
  • replace all occurrences of ASGraphicsCreateImageWithTraitCollectionAndOptions with ASGraphicsCreateImage
  • add new methods with traitCollection parameter to UIImage_ASConvenience and deprecate old ones
  • add traitCollection parameter to asimagenode_modification_block_t (without deprecation)
  • missing tests for UIImage_ASConvenience
  • missing tests for ASImageNodeTintColorModificationBlock
  • don't use deprecated methods in examples and documentation.

@vovasty vovasty force-pushed the vlad/deprecate-ASGraphicsCreateImageWithTraitCollectionAndOptions branch from 4797fed to 04a3091 Compare May 21, 2020 20:55
@vovasty vovasty force-pushed the vlad/deprecate-ASGraphicsCreateImageWithTraitCollectionAndOptions branch from 73b1037 to f58c67d Compare May 22, 2020 18:45
Copy link
Contributor

@bolsinga bolsinga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the tests!

vovasty and others added 16 commits May 22, 2020 12:33
This has proven hard to maintain, and it has limited benefit now that
our CHANGELOG points to GitHub commit ranges.
…tureGroup#1832)

- I tried to make these tests work with OCMock, but that only ever saw 1 call.
- All these do is count `-reloadData` calls. I'm not sure why so many other methods are recorded.
- Why we are doing this is the library targets iOS 5 and is flagged as a warning in builds.
…AndOptions' of github.com:vovasty/Texture into vlad/deprecate-ASGraphicsCreateImageWithTraitCollectionAndOptions
@vovasty vovasty closed this May 22, 2020
@vovasty vovasty deleted the vlad/deprecate-ASGraphicsCreateImageWithTraitCollectionAndOptions branch May 22, 2020 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants