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

Remove fixed IBC drop name #33358

Merged
merged 1 commit into from
Mar 7, 2019
Merged

Remove fixed IBC drop name #33358

merged 1 commit into from
Mar 7, 2019

Conversation

tmat
Copy link
Member

@tmat tmat commented Feb 13, 2019

This will cause the build to pick up the latest.

@tmat
Copy link
Member Author

tmat commented Feb 13, 2019

@jasonmalinowski FYI: This change shouldn't flow to master.

@jasonmalinowski
Copy link
Member

@tmat Is there something we can do inside of the yaml to condition it based on branch name? Git gives no way to stop this to flow short of Tiger dealing with this manually. Tagging @RikkiGibson and his temporary backup @agocke.

@tmat
Copy link
Member Author

tmat commented Feb 13, 2019

@jasonmalinowski Yes, that sounds better.

@jmarolf Should we also make this conditional on branch being built?

VisualStudio.ChannelName: 'int.d16.0stg'

@tmat
Copy link
Member Author

tmat commented Feb 13, 2019

@jasonmalinowski Let's use this PR now to get RPS tests. Meanwhile I can prepare another change that selects the branch conditionally.

@@ -60,7 +60,7 @@ jobs:
-officialSkipTests $(SkipTests)
-officialSkipApplyOptimizationData $(SkipApplyOptimizationData)
/p:RepositoryName=$(Build.Repository.Name)
/p:VisualStudioIbcSourceBranchName="master-vs-deps"
/p:VisualStudioIbcSourceBranchName="dev16.0-vs-deps"
Copy link
Member

Choose a reason for hiding this comment

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

Won't this evetnually break the YAML file in master when it flows back?

@tmat tmat requested review from a team as code owners February 14, 2019 17:31
@tmat tmat changed the title Remove fixed IBC drop name WIP: Remove fixed IBC drop name Feb 15, 2019
@tmat tmat force-pushed the dev/tmat/newibc-d16.0 branch from 9d4358f to b62cad4 Compare February 20, 2019 20:13
@tmat tmat force-pushed the dev/tmat/newibc-d16.0 branch from b62cad4 to da90de3 Compare February 22, 2019 18:18
@tmat tmat changed the title WIP: Remove fixed IBC drop name Remove fixed IBC drop name Mar 7, 2019
@tmat tmat merged commit 02b0fc8 into dev16.0 Mar 7, 2019
@tmat tmat deleted the dev/tmat/newibc-d16.0 branch March 7, 2019 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants