-
Notifications
You must be signed in to change notification settings - Fork 0
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
Create react-native-ci.yml #16
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small mistake here
on: | ||
push: | ||
branches: | ||
- MaryJabro # Temporary for testing: Adjust to main before pushing to main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops looks like you missed this. need to change this to main
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops looks like you missed this. need to change this to main
updating it to main right now, had it as MaryJabro for testing
- MaryJabro # Temporary for testing: Adjust to main before pushing to main | ||
pull_request: | ||
branches: | ||
- MaryJabro #Temporary for testing: Adjust to main before pushing to main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above
Another thing is when this eventually gets approved lets make sure we squash and merge since you have 11 commits. The reason we want to do this is because we don't want to pollute the main branch with a bunch of commits that don't make much sense. Also if something goes wrong we can pinpoint the commit that caused it easier. |
Looks good to me now. Approved |
Added GitHub Actions workflow for Android APK build and testing. Needs reviewal to ensure correctness