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

[old]Add QuizUp to fastlane examples #26

Closed
wants to merge 3 commits into from
Closed

[old]Add QuizUp to fastlane examples #26

wants to merge 3 commits into from

Conversation

alliannas
Copy link
Contributor

@alliannas alliannas commented May 10, 2016

Let me just first say, thank you so much for creating this amazing tool. It has saved us countless hours when doing submissions to Apple!

So to give you a little context on why our setup is like its here, is a little explanation on how we do things.

QuizUp fastlane setup

For Development

We use a feature branch model and when a feature is merged in (or a part of it, if its large we split it up into multiple PRs) our CI automatically builds the hourly_build lane and sends to our internal hockey app.

Each night, if there were any changes done to master we upload a build to our hockey beta group.
This is the daily_build lane.

Translations

QuizUp is currently supported in 7 languages, and its not possible for us to have a very detailed as many of we the features we do are behind an AB tests.
Our global champions create in advance a list of funny changelogs for each locale and our CI tool selects 1 of them for each release to the App Store.

To accomplish this we use 2 custom actions as explained below.

Release flow

We have two main branches, release and master. All work is done against the master branch and all app store releases are done on the release branch.

While waiting for review we still continue to merge new and potentially untested code into master.

To test the App Store build we use Testflight and if a critical bug is found in that build we fix that on the release branch do call the resubmit, which only updates the build number.

Custom actions

We have a lot of custom actions for things we have to do, like selecting the changelogs, updating translations with our internal tool uhura and a few minors ones.


All feedback is greatly appreciated and we look forward to hear from you and hope we can provide some value with open sourcing our fastfile.

QuizUp iOS Team.

@KrauseFx KrauseFx self-assigned this May 10, 2016
@alliannas
Copy link
Contributor Author

Sup? :)

### [QuizUp](https://www.quizup.com)
[![QuizUp](Logos/QuizUp.png)](https://www.quizup.com)

:rocket: **Advanced - Multiple Environments**
Copy link
Member

Choose a reason for hiding this comment

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

The rocket emoji is already taken by fastlane 😉

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh noes, will change :)

@KrauseFx
Copy link
Member

KrauseFx commented Jul 4, 2016

So sorry I somehow lost this tab - thank you so much for this pull request, this looks great already. I added some comments before it's ready to be merged 👍

@KrauseFx KrauseFx removed their assignment Jul 4, 2016
@alliannas
Copy link
Contributor Author

Thanks for taking the time to review @KrauseFx ,I think I've addressed all of your comments :)


s3 = Aws::S3::Resource.new
file_path = options[:ipa]
obj = s3.bucket(ENV['SECRET-PROJECT-BUCKET-NAME']).object(File.basename(file_path))
Copy link
Member

Choose a reason for hiding this comment

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

This should be an option defined in the available_options

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@KrauseFx sorry for taking so long to fix this small thing, I forgot I had this PR open :(

Copy link
Contributor

@dcordero dcordero left a comment

Choose a reason for hiding this comment

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

Looks great to me, just some minor comments added here and there... Thanks for sharing @alliannas

submit_ipa
end

desc "Resubmits the the build"
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor typo here

- desc "Resubmits the the build"
+ desc "Resubmits the build"

desc "- Installs cocoapods"
desc "- Runs tests"
desc "- Cleanup"
lane :test do
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing cocoapods here attending to the description?

- Project setup
- Version Bump & Git Actions
- Slack Notifications
- Custom action for internal tranlsation tools
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor typo:

- Custom action for internal tranlsation tools
+ Custom action for internal translation tools

@alliannas alliannas changed the title Add QuizUp to fastlane examples [old]Add QuizUp to fastlane examples Jan 17, 2017
@alliannas
Copy link
Contributor Author

@dcordero I lost access to the forked repo and had to do a new PR.

The newest comment there addresses typos: #41

@alliannas alliannas closed this Jan 17, 2017
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