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 CI tests #136

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix CI tests #136

wants to merge 2 commits into from

Conversation

emakryo
Copy link

@emakryo emakryo commented Jan 7, 2024

Hi,
I use this rust-android-gradle plugin in every project with Android and Rust. So I hope I can contribute to this plugin.

I noticed that the CI tests failed in my fork. There are two reasons for this:

  • GitHub Actions' runner image update (NDKs)
  • Rust compatibility to NDK has changed (announcement)

I fixed these problems by

  • updating NDK versions used in the test and adding manual installation steps for old NDKs in the CI workflow
  • adding rust channel (toolchain) configuration in tests depending on NDK versions

Also, I update several actions to erase workflow warnings.

@emakryo emakryo closed this Jan 29, 2024
@emakryo emakryo deleted the fix-test branch January 29, 2024 07:05
@emakryo emakryo restored the fix-test branch January 29, 2024 08:06
@emakryo emakryo reopened this Jan 29, 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.

1 participant