Skip to content

Fastlane #125

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

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Fastlane #125

wants to merge 10 commits into from

Conversation

tinypell3ts
Copy link

@tinypell3ts tinypell3ts commented May 1, 2020

Proposed changes

This PR adds a Fastlane flag for installing Fastlane on your machine and adds a fastlane folder to the project root.

The initial setup includes 4 lanes:
test_setup - To ensure Fastlane is setup correct.
version_bump - bump the version and commit to a Github repository.
github_release - builds an iOS and Android version, creates a release and upload assets.
firebase_release - uploads an .ipa and .apk to a chosen firebase project.

Notes

There is room to do so much more, however, I feel this is a good starting point. I split out the beta release lane into three separate lanes. This allows you to use them individually depending on your requirements.

@tinypell3ts tinypell3ts marked this pull request as ready for review May 1, 2020 07:19
@tinypell3ts tinypell3ts requested review from craigcoles and benjaminreid and removed request for craigcoles May 1, 2020 07:19
@tinypell3ts tinypell3ts self-assigned this May 1, 2020
@tinypell3ts tinypell3ts requested a review from craigcoles May 1, 2020 07:19
Copy link
Contributor

@benjaminreid benjaminreid left a comment

Choose a reason for hiding this comment

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

Spot on 🌮

@craigcoles
Copy link
Contributor

@andykitt Can we add some notes to the readme about this also? 🙌

puts "-------------------------------------------------------"
firebase_app_distribution(
app: ENV['IOS_APP_ID'],
ipa_path: "builds/v#{version}/#{appname}.ipa",
Copy link
Author

Choose a reason for hiding this comment

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

This path would fail if you didn't run thegithub_release lane first. Needs a re-think.

Copy link
Author

Choose a reason for hiding this comment

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

I've now added a skip_build option. This means the iOS version will re-build unless you add the skip_build parameter. yarn run firebase:release skip_build:true.

puts "-------------------------------------------------------"
firebase_app_distribution(
app: ENV['ANDROID_APP_ID'],
apk_path: "android/app/build/outputs/apk/debug/app-debug.apk",
Copy link
Author

Choose a reason for hiding this comment

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

This needs to be release not debug.

@craigcoles
Copy link
Contributor

@KateHoward10 Did you manage to have a look at this yesterday? What was the consensus?

@KateHoward10
Copy link
Contributor

@KateHoward10 Did you manage to have a look at this yesterday? What was the consensus?

I did indeed. I haven't actually managed to get a release to work yet, but due to the way I've set up a RN project to test it on, rather than the changes in here. Fastlane got installed pretty painlessly 👍

* master:
  📝 Update references to romulus-cli from previous project codename
  Remove original issue template
  Update issue templates
  Bump version
  React Query Option (#128)
  Bump lodash from 4.17.15 to 4.17.19
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.

4 participants