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

fix(Version): Fix mistica version and catalog version #209

Merged
merged 2 commits into from
Aug 16, 2022

Conversation

jmpg93
Copy link
Contributor

@jmpg93 jmpg93 commented Aug 12, 2022

Hi there!

I'm fixing the mistica catalog version and mistica version.
I'm also updating the script to update version which seems to be working wrong.

@jmpg93 jmpg93 requested a review from a team August 12, 2022 08:30
@jmpg93 jmpg93 self-assigned this Aug 12, 2022
@jmpg93 jmpg93 requested review from DavidMarinCalleja and jmbrocal and removed request for a team August 12, 2022 08:30
@jmpg93 jmpg93 marked this pull request as ready for review August 12, 2022 08:31
@jmpg93
Copy link
Contributor Author

jmpg93 commented Aug 12, 2022

Sorry @DavidMarinCalleja, I've stale your review. I've added dark mode assets as requested by @yceballost

@@ -95,7 +95,7 @@ struct Searchable: ViewModifier {

extension UIImage {
func bordered(borderWidth: CGFloat = 1, color: UIColor) -> UIImage {
UIGraphicsBeginImageContext(size)
UIGraphicsBeginImageContextWithOptions(size, false, UIScreen.main.scale)
Copy link
Contributor

Choose a reason for hiding this comment

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

Just out of curiosity, why do we need to set this option (UIScreen.main.scale) now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because we need to take the scale into account for core graphics to render the image properly. I didn't notice until installing in a real device.

@jmbrocal jmbrocal merged commit a3aaffa into main Aug 16, 2022
@jmbrocal jmbrocal deleted the fix/mistica-catalog branch August 16, 2022 08:10
tuentisre pushed a commit that referenced this pull request Aug 31, 2022
## [18.0.1](v18.0.0...v18.0.1) (2022-08-31)

### Bug Fixes

* **Assets:** IOS-7644 Exclude common assets from  protocol ([#211](#211)) ([e664ecc](e664ecc))
* **SwiftUI:** Fix navigation bar back tint color for iOS 16 ([7eca013](7eca013))
* **Version:** Fix mistica version and catalog version ([#209](#209)) ([a3aaffa](a3aaffa))
@tuentisre
Copy link
Collaborator

🎉 This PR is included in version 18.0.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants