Skip to content

Adding initial sample project - #27

Merged
KevinSchildhorn merged 12 commits into
mainfrom
AddingSampleProject
Sep 1, 2026
Merged

Adding initial sample project#27
KevinSchildhorn merged 12 commits into
mainfrom
AddingSampleProject

Conversation

@KevinSchildhorn

Copy link
Copy Markdown
Contributor

Adding small sample project that adds monocypher, a crypto library.

Base automatically changed from UpdatingReadme to main August 12, 2026 17:08
Comment thread sample/README.md Outdated
Comment thread sample/gradle/wrapper/gradle-wrapper.properties
Comment thread sample/build.gradle.kts Outdated
Comment thread .gitignore Outdated
Comment thread sample/src/nativeTest/kotlin/co/touchlab/cklib/sample/SecureNoteTest.kt Outdated
@KevinSchildhorn

Copy link
Copy Markdown
Contributor Author

I also updated the kotlin and gradle versions that were causing build issues with newer versions of Android Studio and JDK

@samhill303

Copy link
Copy Markdown
Member

I also updated the kotlin and gradle versions that were causing build issues with newer versions of Android Studio and JDK

What issues were you encountering? Seems like it should maybe be a separate PR. I'd also caution against raising the Kotlin version higher than necessary in case it causes compatibility issues for consumers

@KevinSchildhorn

KevinSchildhorn commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

I also updated the kotlin and gradle versions that were causing build issues with newer versions of Android Studio and JDK

What issues were you encountering? Seems like it should maybe be a separate PR. I'd also caution against raising the Kotlin version higher than necessary in case it causes compatibility issues for consumers

I reverted those changes. Versions should be the same now

Comment thread sample/build.gradle.kts Outdated
Comment on lines +58 to +65
// No compilerArgs. Language.C compiles with a hardcoded
// `-std=gnu11 -O3 -Wall -Wextra -Werror`, and both Monocypher 4.0.3 and secure_random.c
// build clean under it, so no -Wno-error= escape hatch is needed.
// compilerArgs.addAll(
// listOf(
// )
// )
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

do we need to keep this commented out portion?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No we can remove that

@KevinSchildhorn
KevinSchildhorn merged commit 2e8da86 into main Sep 1, 2026
@KevinSchildhorn
KevinSchildhorn deleted the AddingSampleProject branch September 1, 2026 13:29
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.

3 participants