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

Add REACT_NATIVE_DOWNLOADS_DIR to specify custom download location #3552

Merged
merged 1 commit into from
Sep 8, 2022

Conversation

Kudo
Copy link
Contributor

@Kudo Kudo commented Sep 7, 2022

Description

it's not ideal to download 3rd party libraries multiple times, so i'm proposing to add REACT_NATIVE_DOWNLOADS_DIR as react-native to specify a custom download locations. also having a pr for expo: expo/expo#19015

Changes

add REACT_NATIVE_DOWNLOADS_DIR environment variable to specify custom download location

Test code and steps to reproduce

execute REACT_NATIVE_DOWNLOADS_DIR=$HOME/foo ./gradlew :app:assembleDebug in Example app and verify whether $HOME/foo contains the download files after builds

Checklist

  • Included code example that can be used to test this change
  • Updated TS types
  • Added TS types tests
  • Added unit / integration tests
  • Updated documentation
  • Ensured that CI passes

@Kudo Kudo marked this pull request as ready for review September 7, 2022 04:50
Copy link
Member

@tomekzaw tomekzaw left a comment

Choose a reason for hiding this comment

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

Thanks a lot for this PR!

Copy link
Member

@piaskowyk piaskowyk left a comment

Choose a reason for hiding this comment

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

🚀

@piaskowyk piaskowyk merged commit 97714b7 into software-mansion:main Sep 8, 2022
@Kudo Kudo deleted the add-custom-download branch September 8, 2022 15:13
piaskowyk pushed a commit that referenced this pull request Oct 6, 2022
…3552)

## Description

it's not ideal to download 3rd party libraries multiple times, so i'm
proposing to add `REACT_NATIVE_DOWNLOADS_DIR` as react-native to specify
a custom download locations. also having a pr for expo:
expo/expo#19015

## Changes

add `REACT_NATIVE_DOWNLOADS_DIR` environment variable to specify custom
download location

## Test code and steps to reproduce

execute `REACT_NATIVE_DOWNLOADS_DIR=$HOME/foo ./gradlew
:app:assembleDebug` in Example app and verify whether $HOME/foo contains
the download files after builds

## Checklist

- [ ] Included code example that can be used to test this change
- [ ] Updated TS types
- [ ] Added TS types tests
- [ ] Added unit / integration tests
- [ ] Updated documentation
- [x] Ensured that CI passes
mthahzan added a commit to mthahzan/react-native-mmkv that referenced this pull request Oct 8, 2022
Added support for custom download directory using `REACT_NATIVE_DOWNLOADS_DIR` environment variable. This is copied from software-mansion/react-native-reanimated#3552.

Possibly fixes mrousavy#436 and mrousavy#455
mrousavy pushed a commit to mrousavy/react-native-mmkv that referenced this pull request Oct 10, 2022
Added support for custom download directory using `REACT_NATIVE_DOWNLOADS_DIR` environment variable. This is copied from software-mansion/react-native-reanimated#3552.

Possibly fixes #436 and #455
fluiddot pushed a commit to wordpress-mobile/react-native-reanimated that referenced this pull request Jun 5, 2023
…oftware-mansion#3552)

## Description

it's not ideal to download 3rd party libraries multiple times, so i'm
proposing to add `REACT_NATIVE_DOWNLOADS_DIR` as react-native to specify
a custom download locations. also having a pr for expo:
expo/expo#19015

## Changes

add `REACT_NATIVE_DOWNLOADS_DIR` environment variable to specify custom
download location

## Test code and steps to reproduce

execute `REACT_NATIVE_DOWNLOADS_DIR=$HOME/foo ./gradlew
:app:assembleDebug` in Example app and verify whether $HOME/foo contains
the download files after builds

## Checklist

- [ ] Included code example that can be used to test this change
- [ ] Updated TS types
- [ ] Added TS types tests
- [ ] Added unit / integration tests
- [ ] Updated documentation
- [x] Ensured that CI passes
yuriisalo123 pushed a commit to yuriisalo123/React-native-mmkv that referenced this pull request Jun 28, 2023
Added support for custom download directory using `REACT_NATIVE_DOWNLOADS_DIR` environment variable. This is copied from software-mansion/react-native-reanimated#3552.

Possibly fixes #436 and #455
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.

None yet

3 participants