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

Add publish data for d17p1 branch #52510

Merged
merged 1 commit into from
Apr 8, 2021
Merged

Conversation

dibarbet
Copy link
Member

@dibarbet dibarbet commented Apr 8, 2021

No description provided.

@dibarbet dibarbet requested a review from JoeRobich April 8, 2021 19:52
@dibarbet dibarbet requested a review from a team as a code owner April 8, 2021 19:52
"version": "4.0.*",
"packageFeeds": "default",
"channels": [],
"vsBranch": "main",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"vsBranch": "main",
"vsBranch": "feature/d17initial",

Copy link
Member Author

Choose a reason for hiding this comment

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

I think this branch is only used in partner previews, by the time we reach p2 I think this branch will no longer be used. So should we just set this to main for now?

Copy link
Member

Choose a reason for hiding this comment

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

I need to dig and see what the vsBranch is used for. I feel like it is the optprof related, but since we aren't inserting it isn't likely an issue.

Copy link
Member Author

Choose a reason for hiding this comment

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

This is where I found it being used -

function GetIbcSourceBranchName() {

for ibc source data.

Copy link
Member Author

Choose a reason for hiding this comment

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

I guess I'm not sure if this should use p1 data or main data since neither are merging into this branch.

Copy link
Member

Choose a reason for hiding this comment

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

I imagine @genlu might know.

@dibarbet dibarbet merged commit 315c2e1 into dotnet:main Apr 8, 2021
@ghost ghost added this to the Next milestone Apr 8, 2021
@dibarbet dibarbet deleted the publishdata_d17p1 branch April 8, 2021 23:21
@dibarbet dibarbet modified the milestones: Next, 16.10.P3 Apr 26, 2021
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