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

Hot fix: upgrade Android SDK version for React Native Themis #979

Merged
merged 1 commit into from
Jan 20, 2023

Conversation

radetsky
Copy link
Contributor

Hotfix for the issue #978

@vixentael vixentael changed the title Hot fix: upgrade version of Android SDK Hot fix: upgrade Android SDK version for React Native Themis Jan 19, 2023
@vixentael vixentael added the O-ReactNative ⚛️ ReactNative platform label Jan 19, 2023
@vixentael
Copy link
Contributor

I think we should cherry pick this PR into release/0.14 branch
https://github.com/cossacklabs/themis/pull/970/files

Copy link
Collaborator

@ilammy ilammy left a comment

Choose a reason for hiding this comment

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

React Native: 3-in-1 upgrade headaches!

@ilammy
Copy link
Collaborator

ilammy commented Jan 20, 2023

I think we should cherry pick this PR into release/0.14 branch

That would be neat to get green builds, but it doesn't seem that any of those CI jobs are checking React Native, so I don't think this PR should be blocked by red CI.

For example, "Android emulator" job is unlikely to get fixed by rolling back the base system, and it's more of Android build system rot mitigated with upgrades on master, but without a backup plan for what to do with 0.14 maintenance.

@vixentael
Copy link
Contributor

I don't think this PR should be blocked by red CI

after seeing your comment in #980 I agree with you

@vixentael vixentael merged commit 00f9824 into release/0.14 Jan 20, 2023
@vixentael
Copy link
Contributor

@radetsky please continue with release, once you have time, and with #980 once you have time again :)

@radetsky radetsky deleted the rad-dev branch February 15, 2023 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-ReactNative ⚛️ ReactNative platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants