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

Users/eyma/appcenter distribute fix aliases #9902

Merged
merged 2 commits into from
Mar 21, 2019

Conversation

octopotato
Copy link

  • Remove conflicting aliases from both distributionGroupIds and distributionStoreId that were causing issues with yaml build configuration
  • Bump patch version of AppCenterDistributeV3

mseyma added 2 commits March 21, 2019 10:44
…utionStoreId to not be conflicting

We were using the same aliases for both distributionGroupId and distributionStoreId which caused a bug in the yaml configurations since it wasn't clear which field a value should resolve to.
Copy link

@damccorm damccorm left a comment

Choose a reason for hiding this comment

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

Thanks for the quick turnaround, LGTM. @stephenmichaelf FYI

@octopotato octopotato merged commit 8120c91 into master Mar 21, 2019
@octopotato octopotato deleted the users/eyma/appcenter-distribute-fix-aliases branch March 21, 2019 17:57
octopotato pushed a commit that referenced this pull request Mar 21, 2019
octopotato pushed a commit that referenced this pull request Mar 21, 2019
octopotato pushed a commit that referenced this pull request Mar 22, 2019
…-m148

[AppCenter/Distribute] Cherrypick #9902 to release/m148
octopotato pushed a commit that referenced this pull request Mar 22, 2019
…-m149

[AppCenter/Distribute] Cherrypick #9902 to release/m149
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.

2 participants