Skip to content

[ANDROID][SDK] Fix asset copy logic + Add android tests + Code cleanup#114

Merged
void-memories merged 6 commits into
NimbleEdge:mainfrom
void-memories:naman/android/fix/asset-folder-copy
Jul 15, 2025
Merged

[ANDROID][SDK] Fix asset copy logic + Add android tests + Code cleanup#114
void-memories merged 6 commits into
NimbleEdge:mainfrom
void-memories:naman/android/fix/asset-folder-copy

Conversation

@void-memories
Copy link
Copy Markdown
Contributor

Description

Please provide a description of this PR.

Fixes # (issue)

Checklist:

  • I have added tests that prove my fix is effective or that my feature works
  • Has user-facing changes. This may include API or behavior changes and performance improvements, etc

@void-memories void-memories requested a review from a team as a code owner July 11, 2025 12:01
Copy link
Copy Markdown
Contributor

@nrjpoddar nrjpoddar left a comment

Choose a reason for hiding this comment

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

Overall LGTM but I will let @Arpit2601 review in more details

Signed-off-by: Naman Anand <naman.anand@nimbleedgehq.ai>
@void-memories void-memories force-pushed the naman/android/fix/asset-folder-copy branch from 60bcf15 to e51c7b0 Compare July 14, 2025 05:34
Comment thread sdks/android/nimblenet_ktx/src/main/kotlin/dev/deliteai/impl/io/FileUtils.kt Outdated
Comment thread sdks/android/nimblenet_ktx/src/androidTest/assets/misc/add_script.ast Outdated
Comment thread sdks/android/nimblenet_ktx/src/main/kotlin/dev/deliteai/impl/io/FileUtils.kt Outdated
void-memories and others added 3 commits July 15, 2025 12:47
Signed-off-by: Naman Anand <naman.anand@nimbleedgehq.ai>
Signed-off-by: Naman Anand <naman.anand@nimbleedgehq.ai>
Signed-off-by: Puneet Jindal <puneet.jindal@nimbleedgehq.ai>
Comment thread sdks/android/nimblenet_ktx/src/main/kotlin/dev/deliteai/impl/io/FileUtils.kt Outdated
Signed-off-by: Puneet Jindal <puneet.jindal@nimbleedgehq.ai>
@void-memories void-memories merged commit 890a923 into NimbleEdge:main Jul 15, 2025
5 of 8 checks passed
@void-memories void-memories deleted the naman/android/fix/asset-folder-copy branch July 15, 2025 13:59
@nrjpoddar
Copy link
Copy Markdown
Contributor

nrjpoddar commented Jul 16, 2025

Addresses #99

Suryansh1191 pushed a commit to Suryansh1191/deliteAI-public that referenced this pull request Jul 21, 2025
NimbleEdge#114)

* code cleanup
* address pr comments
* make copy asset tests generic
* cosmetics

---------
Signed-off-by: Naman Anand <naman.anand@nimbleedgehq.ai>
Co-authored-by: Puneet Jindal <puneet.jindal@nimbleedgehq.ai>
Signed-off-by: Suryansh Bisen <suryansh.bisen@nimbleedgehq.ai>
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.

4 participants