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: XCode warnings #766

Merged
merged 1 commit into from
Jan 20, 2025
Merged

fix: XCode warnings #766

merged 1 commit into from
Jan 20, 2025

Conversation

kirillzyusko
Copy link
Owner

@kirillzyusko kirillzyusko commented Jan 20, 2025

📜 Description

Fixed warnings during project compilation.

💡 Motivation and Context

Warnings is an indicator of a bad code sign. Let's make sure that we don't have such "code smells" and our codebase don't throw any warnings during compilation 😎

📢 Changelog

iOS

  • removed unused conditional UIView casting.

🤔 How Has This Been Tested?

Tested on CI.

📸 Screenshots (if appropriate):

image

📝 Checklist

  • CI successfully passed
  • I added new mocks and corresponding unit-tests if library API was changed

@kirillzyusko kirillzyusko added 🍎 iOS iOS specific refactor You changed the code but it didn't affect functionality labels Jan 20, 2025
@kirillzyusko kirillzyusko self-assigned this Jan 20, 2025
Copy link
Contributor

📊 Package size report

Current size Target Size Difference
167655 bytes 167661 bytes -6 bytes 📉

@kirillzyusko kirillzyusko marked this pull request as ready for review January 20, 2025 11:55
@kirillzyusko kirillzyusko merged commit ca600a8 into main Jan 20, 2025
15 checks passed
@kirillzyusko kirillzyusko deleted the fix/xcode-warnings branch January 20, 2025 11:56
@kirillzyusko kirillzyusko mentioned this pull request Jan 21, 2025
2 tasks
kirillzyusko added a commit that referenced this pull request Jan 21, 2025
## 📜 Description

Fixed newly discovered warnings.

## 💡 Motivation and Context

In paper example I've seen even more warnings and fixed them as well 🙂
initially these warning were added in
#727

Follow up for
#766

## 📢 Changelog

<!-- High level overview of important changes -->
<!-- For example: fixed status bar manipulation; added new types
declarations; -->
<!-- If your changes don't affect one of platform/language below - then
remove this platform/language -->

### iOS

- fixed 2 warnings in `KeyboardMovementObserver`;

## 🤔 How Has This Been Tested?

Tested manually.

## 📝 Checklist

- [x] CI successfully passed
- [x] I added new mocks and corresponding unit-tests if library API was
changed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍎 iOS iOS specific refactor You changed the code but it didn't affect functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant