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

Official Android Docker for react native #36

Merged
merged 1 commit into from
Nov 26, 2018

Conversation

gengjiawen
Copy link
Member

@gengjiawen gengjiawen commented Oct 5, 2018

Provide an official docker to the community and make the ci easier.

@kelset kelset added the 💡 Proposal This label identifies a proposal label Oct 5, 2018
@kelset
Copy link
Member

kelset commented Oct 5, 2018

👋 @gengjiawen!

If I understand correctly, this would mean maintain like a docker image in the registry/marketplace so that people can just write something like FROM react_native_android:latest in their Dockerfile, right?

Does it mean that we should have a version of the image per each RN release?

@gengjiawen
Copy link
Member Author

Yes, you are right, just import the image and add the build script is enough.

Actually if we have not touch the Dockerfile, there is no need to publish a new one.

But if we hook the publish docker script when we draft a new release, it maybe easier this way by not publishing the docker manually.

@axe-fb
Copy link
Collaborator

axe-fb commented Oct 11, 2018

Does this have to be a part of core react native ? Can it not live separately in react-native-community ?

@gengjiawen
Copy link
Member Author

yes, I think it can move out.

@satya164
Copy link
Member

@gengjiawen lemme know if you want to create a repo in react-native-community. I can help with that.

@axe-fb
Copy link
Collaborator

axe-fb commented Oct 15, 2018

@gengjiawen - Once we have moved it out, can we also delete the images from the core repo ?

@axe-fb
Copy link
Collaborator

axe-fb commented Oct 15, 2018

Should we close this issue since we have a resolution for it ?

@gengjiawen
Copy link
Member Author

gengjiawen commented Oct 15, 2018

Once we have moved it out, can we also delete the images from the core repo ?

@axe-fb Yes, maybe remove some script in package.json, but I need @hramos opinion on this.

Should we close this issue since we have a resolution for it ?

Maybe not for now, I want this to get community know this more and discuss how this pr will go.

@gengjiawen lemme know if you want to create a repo in react-native-community. I can help with that.

@satya164 Not for now, but maybe later, I will let you know if it needs, thanks.

@gengjiawen
Copy link
Member Author

I don't want to maintain two same file in two repo, is the dockerfile used internally at facebook @axe-fb ?

@gengjiawen
Copy link
Member Author

@satya164 Can you create a repo for me, I think I want to try it in community due to facebook/react-native-website#653.

@sibelius
Copy link
Member

https://twitter.com/sseraphini/status/1061919273113829376

We have this docker image https://github.com/entria/docker-react-native-android

we've been using it on some open source projects like react-native-camera and react-native-share

@gengjiawen
Copy link
Member Author

@sibelius The docker is intended to do more then build an react native android app, it also has the ability to build and test react native source code.

@axe-fb
Copy link
Collaborator

axe-fb commented Nov 16, 2018

@gengjiawen - what are the next steps on this proposal ?

@gengjiawen
Copy link
Member Author

@axe-fb I need @hramos opinion on this, to see whether or not move the Dockerfile out of the main repo.

@axe-fb
Copy link
Collaborator

axe-fb commented Nov 26, 2018

@gengjiawen - should we merge this PR, since we will now be acting upon the proposal ?

@gengjiawen
Copy link
Member Author

I think it can merge.

@kelset kelset merged commit a53b3c1 into react-native-community:master Nov 26, 2018
facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Jan 24, 2019
Summary:
Simplify current build and offer it to community as official docker.
I have also created an [rfc](react-native-community/discussions-and-proposals#36).
pass all current ci.
none
 [GENERAL] [INTERNAL] [CI] - migrate ci to official docker
Pull Request resolved: #21477

Differential Revision: D13801696

Pulled By: cpojer

fbshipit-source-id: fa235cf21ab5af2c87eb3c1a14788fab01c71ccf
matt-oakes pushed a commit to matt-oakes/react-native that referenced this pull request Feb 7, 2019
Summary:
Simplify current build and offer it to community as official docker.
I have also created an [rfc](react-native-community/discussions-and-proposals#36).
pass all current ci.
none
 [GENERAL] [INTERNAL] [CI] - migrate ci to official docker
Pull Request resolved: facebook#21477

Differential Revision: D13801696

Pulled By: cpojer

fbshipit-source-id: fa235cf21ab5af2c87eb3c1a14788fab01c71ccf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 Proposal This label identifies a proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants