dotnet / source-build

A repository to track efforts to produce a source tarball of the .NET Core SDK and all its components
MIT License
261 stars 127 forks source link

Cleanup `release/9.0.1xx-teststablep5` Branches #4469

Open ellahathaway opened 2 weeks ago

ellahathaway commented 2 weeks ago

As part of the work to get a coherent build for the release post the installer -> sdk merge, a branch titled release/9.0.1xx-teststablep5 was created in sdk and dotnet. The existence of this branch is causing the source-build-license-scan pipeline to be triggered to run on the branch every Monday. This wastes compute, so, seeing as this branch is no longer in use, we should remove the branch both internally and publicly.

dotnet-issue-labeler[bot] commented 2 weeks ago

I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

dotnet-issue-labeler[bot] commented 2 weeks ago

I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

mthalman commented 1 week ago

This was a temporary branch but there are also other older preview branches that are causing the license scan pipeline to run: release/9.0.1xx-preview4 and release/9.0.1xx-preview5 (internal links). These continue to run because there has not been a successful build of the license scan for those branches. But those branches will not have any new changes.

We have 3 options:

Deleting the branches would be preferable. @mmitche - Can we do that for the VMR since we have release tags? If so, we should probably apply that to 8.0 as since there are still 8.0 preview branches. And also make this part of the regular release workflow.

mmitche commented 1 week ago

Yeah I'm fine with deleting from the VMR