Skip to content

[.github] Remove ArcReview #39505

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.

Sign up for GitHub

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

Jump to bottom
Merged
mikeharder merged 1 commit into from
Jan 8, 2026
Merged

[.github] Remove ArcReview #39505

mikeharder merged 1 commit into from
Jan 8, 2026

Conversation

@mikeharder
Copy link
Member

@mikeharder mikeharder commented Jan 8, 2026 •
edited
Loading

  • Label is defunct and should be removed
  • Left over from a previous time when private Arc clouds each had their own top-level VM resource
  • Since then the design has changed and all private Arc clouds use HybridCompute.machine as the top level resource
  • Removal requested by @edyoung and @jianyunt

@mikeharder
[.github] Remove ArcReview
976f825 
- Label is defunct and should be removed
- Left over from a previous time when private Arc clouds each had their own top-level VM resource
- Since then the design has changed and all private Arc clouds use HybridCompute.machine as the top level resource
@github-actions
Copy link

github-actions bot commented Jan 8, 2026 •
edited
Loading

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ The required check named Protected Files has failed. Refer to the check in the PR's 'Checks' tab for details on how to fix it and consult the aka.ms/ci-fix guide


Comment generated by summarize-checks workflow run.

@mikeharder mikeharder self-assigned this Jan 8, 2026
@mikeharder mikeharder moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 📆🎇 Jan 8, 2026
@edyoung
Copy link

edyoung commented Jan 8, 2026

Thanks!

@mikeharder mikeharder merged commit 1360de6 into Azure:main Jan 8, 2026
60 of 64 checks passed
@mikeharder mikeharder deleted the remove-arcreview branch January 8, 2026 19:03
@kurtzeborn kurtzeborn moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 📆🎇 Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@weshaggard weshaggard Awaiting requested review from weshaggard weshaggard is a code owner

@benbp benbp Awaiting requested review from benbp benbp is a code owner

@jianyunt jianyunt Awaiting requested review from jianyunt

@raosuhas raosuhas Awaiting requested review from raosuhas

1 more reviewer

@edyoung edyoung edyoung approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@mikeharder mikeharder

Labels

None yet

Projects

Azure SDK EngSys 📆🎇
Status: 🎊 Closed

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mikeharder @edyoung