dotnet / maintenance-packages

Repository that hosts packages from the .NET platform whose original home/branch is not building any longer.
MIT License
22 stars 10 forks source link

YML should upload produced packages as pipeline artifacts #164

Open ViktorHofer opened 1 week ago

ViktorHofer commented 1 week ago

We should have packages available in the pipeline run as pipeline artifacts for inspection. This makes it easier to double check PR changes.

carlossanlop commented 1 week ago

I was thinking about this the other day. I noticed we do generate the packages, they just don't get stored in a persistent location.

Will we get any kind of pushback from dnceng for using a lot of new space? We would basically create a new persistent artifact location for every commit in a PR. Unless they get deleted and overriden whenever a new commit is pushed?

ViktorHofer commented 1 week ago

Nope, we do this in every repository. Pipeline artifacts are expected to be used and they a retention policy setting attached anyway. It's perfectly fine to do this.