Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Fixes #1566, #2076 #2194

Merged
merged 1 commit into from
May 2, 2019
Merged

Conversation

mitchhentges
Copy link
Contributor

Note: this changes:

  • The gradle command from ./gradlew assembleReleaseRaptor to ./gradlew assembleRaptor
  • The application ID changes from org.mozilla.fenix to org.mozilla.fenix.raptor (note: --binary is updated as part of this PR to use the new ID)
  • The raptor route changes to have release-raptor replaced with raptor

Pull Request checklist

  • Quality: This PR builds and passes detekt/ktlint checks (A pre-push hook is recommended)
  • Tests: This PR includes thorough tests or an explanation of why it does not
  • Changelog: This PR includes a changelog entry or does not need one
  • Accessibility: The code in this PR follows accessibility best practices or does not include any user facing features

@mitchhentges mitchhentges requested review from a team as code owners May 1, 2019 13:41
@mitchhentges
Copy link
Contributor Author

Associated m-c work

Copy link
Contributor

@JohanLorenzo JohanLorenzo left a comment

Choose a reason for hiding this comment

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

⛵️ it!

routes = []
if self.repo_url == _OFFICIAL_REPO_URL:
routes = [
'index.project.mobile.fenix.v2.branch.master.revision.{}.{}.{}'.format(
self.commit, build_type, architecture
self.commit, 'raptor', architecture
Copy link
Contributor

Choose a reason for hiding this comment

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

Okay to land these breaking changes because appending .raptor to the package name already breaks the harness. Thank you, @mitchhentges, for telling me this offline

@mitchhentges mitchhentges force-pushed the release-raptor-rename branch 2 times, most recently from 97d51c6 to 60c52d1 Compare May 2, 2019 10:07
@boek boek merged commit 9a2cf43 into mozilla-mobile:master May 2, 2019
@mitchhentges mitchhentges deleted the release-raptor-rename branch May 2, 2019 14:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants