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

Set ProducesDotNetReleaseShippingAssets property in Publishing.props #7578

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

MilenaHristova
Copy link
Contributor

Add a boolean property named ProducesDotNetReleaseShippingAssets in Publishing.props for repos that produce .NET shipping packages (packages that we ship with the release infra)
Based on this property we will select which packages to ship as part of .NET on release day.

This is a infrastructure only change. It will add extra metadata to the MergedManifest.xml produced during CI build.

Issue: https://github.com/dotnet/release/issues/822

@MilenaHristova MilenaHristova requested a review from a team as a code owner February 21, 2024 11:06
@MilenaHristova MilenaHristova merged commit aff7a59 into dotnet:main Feb 21, 2024
10 checks passed
@MilenaHristova MilenaHristova deleted the publishing-props branch February 21, 2024 17:40
@MilenaHristova
Copy link
Contributor Author

/backport to release/8.0.1xx

Copy link
Contributor

Started backporting to release/8.0.1xx: https://github.com/dotnet/templating/actions/runs/8062936046

@MilenaHristova
Copy link
Contributor Author

/backport to release/6.0.4xx

Copy link
Contributor

Started backporting to release/6.0.4xx: https://github.com/dotnet/templating/actions/runs/8062954761

@MilenaHristova
Copy link
Contributor Author

/backport to release/8.0.3xx

Copy link
Contributor

Started backporting to release/8.0.3xx: https://github.com/dotnet/templating/actions/runs/8063015976

@MilenaHristova
Copy link
Contributor Author

/backport to release/7*

Copy link
Contributor

Started backporting to release/7: https://github.com/dotnet/templating/actions/runs/8063144029

Copy link
Contributor

@MilenaHristova an error occurred while backporting to release/7, please check the run log for details!

Error: The specified backport target branch release/7 wasn't found in the repo.

@MilenaHristova
Copy link
Contributor Author

/backport to release/8.0.2xx

Copy link
Contributor

Started backporting to release/8.0.2xx: https://github.com/dotnet/templating/actions/runs/8063174336

@MilenaHristova
Copy link
Contributor Author

/backport to release/7.0.3xx

Copy link
Contributor

Started backporting to release/7.0.3xx: https://github.com/dotnet/templating/actions/runs/8063181688

@MilenaHristova
Copy link
Contributor Author

/backport to release/6.0.1xx

Copy link
Contributor

Started backporting to release/6.0.1xx: https://github.com/dotnet/templating/actions/runs/8063187636

@MilenaHristova
Copy link
Contributor Author

/backport to release/7.0.4xx

Copy link
Contributor

Started backporting to release/7.0.4xx: https://github.com/dotnet/templating/actions/runs/8063192926

@MilenaHristova
Copy link
Contributor Author

/backport to release/7.0.1xx

Copy link
Contributor

Started backporting to release/7.0.1xx: https://github.com/dotnet/templating/actions/runs/8063197885

@MilenaHristova
Copy link
Contributor Author

/backport to release/6.0.3xx

Copy link
Contributor

Started backporting to release/6.0.3xx: https://github.com/dotnet/templating/actions/runs/8063204552

@joeloff
Copy link
Member

joeloff commented Feb 27, 2024

@MilenaHristova I approved the back ports, I think we may have to wait for servicing changes (branding and codeflow to complete) before merging this into the release/xyz branches

@MilenaHristova
Copy link
Contributor Author

@MilenaHristova I approved the back ports, I think we may have to wait for servicing changes (branding and codeflow to complete) before merging this into the release/xyz branches

sorry, I merged one of the PRs before I saw this. Why is it a problem to merge it now?

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