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

Update channelName #9378

Merged
merged 1 commit into from
Jan 17, 2024
Merged

Conversation

tmeschter
Copy link
Contributor

@tmeschter tmeschter commented Jan 17, 2024

Update the channel used for the baseline VS build for OptProf collection runs. The latest build of the specified channel is combined with the project system binaries built out of our repo to create the installer used in the run. Since our dev17.9.x branch inserts into the rel/d17.9 branch in the VS repo we need to use the int.d17.9 channel. Using int.main might work for a while, but eventually we're get to the point where our 17.9 builds are no longer compatible with VS main builds and our OptProf test will fail. By updating now we avoid that problem.

Microsoft Reviewers: Open in CodeFlow

Update the channel used for the baseline VS build for OptProf collection runs.
The latest build of the specified channel is combined with the project
system binaries built out of our repo to create the installer used in the
run. Since our `dev17.9.x` branch inserts into the `rel/d17.9` branch in
the VS repo we need to use the `int.d17.9` channel. Using `int.main` might
work for a while, but eventually we're get to the point where our 17.9
builds are no longer compatible with VS `main` builds and our OptProf
test will fail. By updating now we avoid that problem.
@tmeschter tmeschter requested a review from a team as a code owner January 17, 2024 21:34
@tmeschter tmeschter merged commit 26bdea5 into dotnet:dev17.9.x Jan 17, 2024
5 checks passed
@tmeschter tmeschter deleted the 240117-UpdateChannelName branch January 17, 2024 22:44
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