Skip to content

Add task to setup dev dependencies #10864

Merged
loremattei merged 3 commits intodevelopfrom
add-dependencies-script
Nov 29, 2019
Merged

Add task to setup dev dependencies #10864
loremattei merged 3 commits intodevelopfrom
add-dependencies-script

Conversation

@loremattei
Copy link
Copy Markdown
Contributor

This PR adds a gradle task to make it easier to setup some development dependencies, like bundle, fastlane and the release-toolkit that may be useful for credential management, screenshots, releases and so.

To test:

  • If you have it, delete the vendor folder in your repo.
  • Run ./gradlew prodDeps.
  • Verify that bundle installation is checked and that the tool is installed if missing.
  • Verify that all the gems are installed.
  • Verify that the correct credentials are applied.

PR submission checklist:

  • I have considered adding unit tests where possible.

  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@loremattei loremattei added this to the 13.8 milestone Nov 27, 2019
@loremattei loremattei requested a review from jkmassel November 27, 2019 16:20
@peril-wordpress-mobile
Copy link
Copy Markdown

You can test the changes on this Pull Request by downloading the APK here.

Copy link
Copy Markdown
Contributor

@jkmassel jkmassel left a comment

Choose a reason for hiding this comment

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

✅ Verify that bundle installation is checked and that the tool is installed if missing.
✅ Verify that the correct credentials are applied (verified by running be fastlane build_pre_releases)
✅ Verify that all the gems are installed (verified by running be fastlane run configure_apply after doing a test build)

@loremattei loremattei merged commit da08452 into develop Nov 29, 2019
@loremattei loremattei deleted the add-dependencies-script branch November 29, 2019 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants