Skip publishing wixpacks added to the Artifact item group by default #15461
Conversation
This explicitly doesn't touch ItemsToPushToBlobFeed as that item group is undocumented and Artifact items with SkipPublish=true don't even make it into ItemsToPushToBlobFeed, so we'd have to remove items instead, which doesn't give a good user gesture to force publishing wixpacks anyway for some reason.
ViktorHofer
left a comment
ViktorHofer
left a comment
There was a problem hiding this comment.
I have my reservations with this change. The SkipPublish metadata was only added so that all artifact items could be defined in a single place in Sign.proj, even ones that should only be signed but not published. I want to remove that metadata again. I should have used other repo local means to achieve that.
We will eventually update the default artifact globs in Arcade to include zips and will then make sure that wixpack.zip files aren't included.
Until then, I'm ok with this change as the plan is to remove the Artifact extension point anyway.
|
I do personally prefer the Artifact item group more than the |
2 participants
This explicitly doesn't touch ItemsToPushToBlobFeed as that item group is undocumented and Artifact items with SkipPublish=true don't even make it into ItemsToPushToBlobFeed, so we'd have to remove items instead, which doesn't give a good user gesture to force publishing wixpacks anyway for some reason.
To double check: