Skip to content

Conversation

@epananth
Copy link
Member

@epananth epananth commented Sep 2, 2021

To double check:

Issue -> #7818

@epananth epananth requested review from MattGal and riarenas September 2, 2021 21:05
Copy link
Member

@MattGal MattGal left a comment

Choose a reason for hiding this comment

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

Looks correct to my understanding of publishing, I admit I don't understand what "PublishingInfraVersion.Next" really means but following what net5 does looks right.

@epananth
Copy link
Member Author

epananth commented Sep 2, 2021

Publishing.Next means V3 publishing, so from .Net 5 onwards we are using only V3 so we use that enum..

@epananth epananth added the auto-merge Automatically merge PR once CI passes. label Sep 2, 2021
@ghost
Copy link

ghost commented Sep 2, 2021

Hello @epananth!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 00949c5 into dotnet:main Sep 2, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Automatically merge PR once CI passes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants