Skip to content

run it on github action#6

Merged
Siddhesh2377 merged 6 commits intoSiddhesh2377:fresh-newfrom
kkgit2008:fresh-new_03
Sep 7, 2025
Merged

run it on github action#6
Siddhesh2377 merged 6 commits intoSiddhesh2377:fresh-newfrom
kkgit2008:fresh-new_03

Conversation

@kkgit2008
Copy link
Copy Markdown
Contributor

No description provided.

@Siddhesh2377
Copy link
Copy Markdown
Owner

I just got home, I will review them soon

@kkgit2008
Copy link
Copy Markdown
Contributor Author

OK

@Siddhesh2377 Siddhesh2377 merged commit 24c783f into Siddhesh2377:fresh-new Sep 7, 2025
@kkgit2008
Copy link
Copy Markdown
Contributor Author

kkgit2008 commented Sep 8, 2025

Sorry for making trouble. Maybe you're using visual studio to build this project, but the code i pulled doesn't fit Visual studio, it can only run on github action.

Code sdk.dir=*
in file local.properties should not download to visual studio, but it works well on github action.

Code externalNativeBuild { cmake { path("src/main/cpp/CMakeLists.txt") version = "3.22.1" } }
in file ai-core/build.grade can't work on github action, but it's needed by visual studio.

@Siddhesh2377
Copy link
Copy Markdown
Owner

I am using Android studio, and I think I will create a new branch for development and I will deploy changes in GitHub action, just need some time

@kkgit2008
Copy link
Copy Markdown
Contributor Author

That sounds like a solid plan! It's a great practice to keep the main codebase stable.
Looking forward to seeing the changes and the workflow in action once it’s ready!

@Siddhesh2377
Copy link
Copy Markdown
Owner

Yea, I have my exams now in 2 days so that's why I am not committing any patches

@kkgit2008
Copy link
Copy Markdown
Contributor Author

That' fine, exam is the most important

@kkgit2008 kkgit2008 deleted the fresh-new_03 branch September 12, 2025 12:20
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.

2 participants