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 15 build #70

Merged
merged 20 commits into from
Nov 27, 2023
Merged

Fix Xcode 15 build #70

merged 20 commits into from
Nov 27, 2023

Conversation

kyzmitch
Copy link
Owner

@kyzmitch kyzmitch commented Oct 29, 2023

Report #67

  • fixed build on Apple silicon Mac by updating Kotlin Multiplatform to 1.9.20
  • disabled remote Kotlin linter GitHub action completely because it ignores YML config file and local Linter has different rules that remote one from action. CottonBase Kotlin linter still works for now.
  • fixed SwiftLint build step as well?
  • add small amount of Swift 5.9 specific code
  • small Android Compose changes

@kyzmitch kyzmitch closed this Nov 26, 2023
@kyzmitch kyzmitch reopened this Nov 26, 2023
@kyzmitch kyzmitch closed this Nov 26, 2023
@kyzmitch kyzmitch reopened this Nov 26, 2023
@kyzmitch kyzmitch closed this Nov 27, 2023
@kyzmitch kyzmitch reopened this Nov 27, 2023
… ktlint and it also ignores yml config file, so, it is not possible to ignore some lint errors
@kyzmitch kyzmitch closed this Nov 27, 2023
@kyzmitch kyzmitch reopened this Nov 27, 2023
@kyzmitch kyzmitch closed this Nov 27, 2023
@kyzmitch kyzmitch reopened this Nov 27, 2023
@kyzmitch kyzmitch merged commit faefff4 into master Nov 27, 2023
5 checks passed
kyzmitch added a commit that referenced this pull request Mar 13, 2024
* Update Kotlin to 1.9.0 in Multiplatform cotton-base project because latest Android Studio Giraffe uses 1.9. Fix OptIn annotations for new Kotlin version.

* Fix Xcode 15 and add build for Apple Silicon macos for the unit tests

* Fix Xcpretty calls, fix macOS sdk for local make command to just use latest SDK because local version is different on different Apple devices

* Attempt to fix linking

* mocks gen fix

* Update kotlin plugin for Xcode 15

* Add not related swift 5.9 feature for AsyncStream type

* Move some files to different build target where they belong

* Fix build on Apple silicon

* Attempt to select Xcode version on the runner

* Android gradle plugin fix, 7.4.0 is the maximum version supported by IntelliJ IDEA

* JDK 17

* Use corretto-17 jdk

* Implement top site selection

* State hoisting?

* Store vm on the upper level

* Fix kotlin lint errors

* Disable ktlint github action because it conflicts too much with local ktlint and it also ignores yml config file, so, it is not possible to ignore some lint errors

* Remove android linter step completely
@kyzmitch kyzmitch deleted the bug/xcode_15_and_kotlin_1_9_build branch March 13, 2024 09:01
kyzmitch added a commit that referenced this pull request Mar 13, 2024
* Update Kotlin to 1.9.0 in Multiplatform cotton-base project because latest Android Studio Giraffe uses 1.9. Fix OptIn annotations for new Kotlin version.

* Fix Xcode 15 and add build for Apple Silicon macos for the unit tests

* Fix Xcpretty calls, fix macOS sdk for local make command to just use latest SDK because local version is different on different Apple devices

* Attempt to fix linking

* mocks gen fix

* Update kotlin plugin for Xcode 15

* Add not related swift 5.9 feature for AsyncStream type

* Move some files to different build target where they belong

* Fix build on Apple silicon

* Attempt to select Xcode version on the runner

* Android gradle plugin fix, 7.4.0 is the maximum version supported by IntelliJ IDEA

* JDK 17

* Use corretto-17 jdk

* Implement top site selection

* State hoisting?

* Store vm on the upper level

* Fix kotlin lint errors

* Disable ktlint github action because it conflicts too much with local ktlint and it also ignores yml config file, so, it is not possible to ignore some lint errors

* Remove android linter step completely
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.

1 participant