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

Publish to multiple channels -Part 1 #1502

Merged
merged 1 commit into from
Jan 8, 2021
Merged

Publish to multiple channels -Part 1 #1502

merged 1 commit into from
Jan 8, 2021

Conversation

epananth
Copy link
Member

@epananth epananth commented Jan 5, 2021

Publish to multiple channels -> dotnet/arcade#6669

Since comma does not work, using hyphen instead.

@epananth epananth changed the title Publish to multiple channels Publish to multiple channels -Part 1 Jan 7, 2021
@epananth epananth marked this pull request as ready for review January 7, 2021 03:43
@epananth epananth requested review from mmitche and riarenas January 7, 2021 03:44
@riarenas
Copy link
Member

riarenas commented Jan 7, 2021

Do you have a test build where I can see this working?

@epananth
Copy link
Member Author

epananth commented Jan 7, 2021

Ran a build locally -> https://dnceng.visualstudio.com/internal/_build/results?buildId=941779&view=logs&j=ba23343f-f710-5af9-782d-5bd26b102304&t=6e277ba4-1c1e-552d-b96f-db0aeb4be20a

It will fail cos it was not able to restore the arcade version which is weird. But I am able to test the channel is bifurcated with hyphen.

@riarenas
Copy link
Member

riarenas commented Jan 7, 2021

There was a widespread azure artifacts incident. You should be able to run it again

@epananth
Copy link
Member Author

epananth commented Jan 7, 2021

I retried it twice, no luck :( Prob will have to wait and try again..

@riarenas
Copy link
Member

riarenas commented Jan 7, 2021

https://pkgs.dev.azure.com/dnceng/internal/_packaging/general-testing-internal/nuget/v3/index.json is not a valid feed, I think the one you're looking for is https://pkgs.dev.azure.com/public/_packaging/general-testing-internal/nuget/v3/index.json

EDIT: Actually nevermind, I was misreading the logs.

@epananth
Copy link
Member Author

epananth commented Jan 8, 2021

@epananth
Copy link
Member Author

epananth commented Jan 8, 2021

I think this is ready for approval now :)

@epananth epananth self-assigned this Jan 8, 2021
@epananth epananth merged commit 1346ed1 into master Jan 8, 2021
@riarenas riarenas deleted the test-multi-channel branch May 13, 2022 17: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