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

Remove stale PublishToBlob.proj #8727

Merged
merged 1 commit into from
May 3, 2023
Merged

Conversation

rainersigwald
Copy link
Member

As far as I know this file is no longer required by Arcade.

@Forgind
Copy link
Member

Forgind commented May 2, 2023

Is there an easy way to validate no longer required by Arcade?

@JanKrivanek
Copy link
Member

Is there an easy way to validate no longer required by Arcade?

Based on dotnet/arcade#1561 and following changes it seems it was replaced with PushToAzureDevOpsArtifacts
Moreover - it's not defined/called/imported anywhere in current Arcade code.

(That being said - I totally support descriptions that require less digging for context :-))

@rainersigwald
Copy link
Member Author

I couldn't find any mention of it in https://github.com/dotnet/arcade or in our repo, but I don't know of a way to fully validate other than going forward with an official build.

@rainersigwald rainersigwald added merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now. merge-carefully Merge one at a time, verifying this doesn't break anything down the chain. labels May 3, 2023
@Forgind Forgind merged commit 6882ab9 into dotnet:main May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-carefully Merge one at a time, verifying this doesn't break anything down the chain. merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants