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

Version Updates #39

Merged
merged 3 commits into from
Jan 13, 2024
Merged

Version Updates #39

merged 3 commits into from
Jan 13, 2024

Conversation

vanshg
Copy link
Contributor

@vanshg vanshg commented Dec 14, 2023

Gradle to 8.5 (using ./gradlew wrapper --gradle-version=8.5)
Kotlin to 1.9.10 (dokka is incompatible with 1.9.21 at the moment) AGP to 8.2
AndroidX Core to 1.12
CameraX to 1.3.1
Compose to 2023.10.01

Gradle to 8.5
Kotlin to 1.9.10 (dokka is incompatible with 1.9.21 at the moment)
AGP to 8.2
AndroidX Core to 1.12
CameraX to 1.3.1
Compose to 2023.10.01
Copy link
Owner

@ujizin ujizin left a comment

Choose a reason for hiding this comment

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

Hey! First of all thanks so much for supporting Camposer 🙏

I reviewed some points here that's affecting the library and sample app.

There's another one, If possible could you please update the compilerSdk and targetSdk to 34? Otherwise the library won't compile

Later, I'll update the video feature and bump the cameraX version.

Thanks again! Have a good weekend 😄

gradle/libs.versions.toml Outdated Show resolved Hide resolved
coil = "2.2.2"
androidx-core = "1.12.0"
androidx-test-rules = "1.5.0"
camerax = "1.3.1"
Copy link
Owner

Choose a reason for hiding this comment

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

There're some breaking changes on video feature in this version that needs to resolve 😢

I will try to check this later, but please feel free to resolve or rollback to 1.2.3 again 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I see you opened up a PR to address this. I rolled back the version here, but lmk if you want me to keep it bumped for easier merging into your branch

Copy link
Owner

Choose a reason for hiding this comment

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

Hi! Thanks for clarifying things for me!

I've updated, and soon I'll create a new release with our changes. Thanks for your contributions and patience on this.

gradle/libs.versions.toml Outdated Show resolved Hide resolved
androidx-test-rules = "1.5.0"
camerax = "1.3.1"
compose-bom = "2023.10.01"
compose-compiler = "1.5.6"
Copy link
Owner

Choose a reason for hiding this comment

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

Forgot about this one. The compiler needs to be 1.5.3 because of the kotlin compat

ref: https://developer.android.com/jetpack/androidx/releases/compose-kotlin

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, missed that (I updated this first and then realized I needed to keep Kotlin at 1.9.10 due to dokka but didn't change the compiler version to accomodate)

@vanshg vanshg requested a review from ujizin December 22, 2023 19:26
Copy link
Owner

@ujizin ujizin left a comment

Choose a reason for hiding this comment

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

LGTM

@ujizin ujizin merged commit f5ea497 into ujizin:main Jan 13, 2024
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

2 participants