Skip to content

Remove src/arcade/eng/common from repos that took it #4837

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

Open
dkurepa opened this issue May 14, 2025 · 6 comments
Open

Remove src/arcade/eng/common from repos that took it #4837

dkurepa opened this issue May 14, 2025 · 6 comments

Comments

@dkurepa
Copy link
Member

dkurepa commented May 14, 2025

Because of #4751 (comment), there was a number of repos that took a dependency update from the VMR and accidentally added src/arcade/eng/common https://github.com/search?q=org%3Adotnet%20path%3Asrc%2Farcade%2Feng%2Fcommon%2FREADME.md&type=code. Once we have the fix rolled out, we should make sure to remove all of these

@akoeplinger
Copy link
Member

Also one hit in the microsoft org: https://github.com/search?q=org%3Amicrosoft+path%3Asrc%2Farcade%2Feng%2Fcommon%2FREADME.md&type=code

It's possible it also happened for servicing branches right? GitHub search won't find those.

@dkurepa
Copy link
Member Author

dkurepa commented May 15, 2025

the fix for the bug that caused this has been rolled out, We should now go through the repos that merged these PRs and remove the arcade src/arcade/eng/common

@dkurepa
Copy link
Member Author

dkurepa commented May 15, 2025

Unmerged PRs which should be closed:

PRs to remove src/arcade/eng/common:

@akoeplinger
Copy link
Member

akoeplinger commented May 15, 2025

@akoeplinger
Copy link
Member

akoeplinger commented May 15, 2025

So turns out just deleting src/arcade might not be enough, we also need to replace eng/common with a good copy because when the wrong folder was added git recognized some of the files as "moved" there in some repos like wcf.

@akoeplinger
Copy link
Member

Found an easier way: trigger the arcade subscription, the new PR will put the files back into eng/common and then you can just delete src/arcade in the PR.

akoeplinger added a commit to dotnet/fsharp that referenced this issue May 15, 2025
akoeplinger added a commit to akoeplinger/arcade-validation that referenced this issue May 15, 2025
akoeplinger added a commit to akoeplinger/install-scripts that referenced this issue May 15, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to akoeplinger/metadata-tools that referenced this issue May 15, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
T-Gro pushed a commit to dotnet/fsharp that referenced this issue May 16, 2025
* Update dependencies from https://github.com/dotnet/arcade build 20250513.5

Microsoft.DotNet.Arcade.Sdk
 From Version 9.0.0-beta.25255.5 -> To Version 9.0.0-beta.25263.5

* Delete wrong src/arcade folder

See dotnet/arcade-services#4837

---------

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Alexander Köplinger <[email protected]>
ViktorHofer pushed a commit to dotnet/arcade-validation that referenced this issue May 19, 2025
akoeplinger added a commit to dotnet/templating that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to akoeplinger/arcade-validation that referenced this issue May 19, 2025
akoeplinger added a commit to dotnet/aspnetcore that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/cecil that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/dotnet-monitor that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/cecil that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/installer that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/sourcelink that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/windowsdesktop that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/xharness that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/efcore that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
ViktorHofer pushed a commit to dotnet/arcade-validation that referenced this issue May 19, 2025
@akoeplinger akoeplinger self-assigned this May 19, 2025
AndriySvyryd pushed a commit to dotnet/efcore that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/xharness that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
dotnet-maestro bot added a commit to dotnet/dotnet-monitor that referenced this issue May 19, 2025
[feature/10.0] Update dependencies from dotnet/dotnet


 - Delete src/arcade/eng/common

It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to akoeplinger/arcade-validation that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/installer that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/extensions-samples that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/wpf that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
ViktorHofer pushed a commit to dotnet/arcade-validation that referenced this issue May 19, 2025
ViktorHofer pushed a commit to dotnet/extensions-samples that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
akoeplinger added a commit to dotnet/wpf that referenced this issue May 19, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
v-wuzhai pushed a commit to dotnet/sdk that referenced this issue May 20, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837

/cc @v-wuzhai please also delete this folder when merging up this branch
ViktorHofer added a commit to dotnet/templating that referenced this issue May 20, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
ViktorHofer added a commit to dotnet/templating that referenced this issue May 20, 2025
* Update dependencies from https://github.com/dotnet/arcade build 20250425.5

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.25214.7 -> To Version 8.0.0-beta.25225.5

* Update dependencies from https://github.com/dotnet/arcade build 20250428.2

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.25214.7 -> To Version 8.0.0-beta.25228.2

* Update dependencies from https://github.com/dotnet/arcade build 20250429.5

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.25214.7 -> To Version 8.0.0-beta.25229.5

* Update dependencies from https://github.com/dotnet/arcade build 20250430.1

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.25214.7 -> To Version 8.0.0-beta.25230.1

* Update branding to 8.0.117

* Update queue to ubuntu 2204

* [release/8.0.3xx] Update dependencies from dotnet/arcade (#9014)

[release/8.0.3xx] Update dependencies from dotnet/arcade


 - Use System.Text.Json in tests instead of unlisted log4net

* [release/8.0.1xx] Update dependencies from dotnet/arcade (#9013)

[release/8.0.1xx] Update dependencies from dotnet/arcade


 - Use System.Text.Json in tests instead of unlisted log4net

* Update dependencies from https://github.com/dotnet/arcade build 20250514.2 (#9024)

[release/8.0.1xx] Update dependencies from dotnet/arcade

* Update dependencies from https://github.com/dotnet/arcade build 20250514.2 (#9025)

[release/8.0.3xx] Update dependencies from dotnet/arcade

* Update dependencies from https://github.com/dotnet/arcade build 20250514.2 (#9026)

[release/8.0.4xx] Update dependencies from dotnet/arcade

* Update branding to 8.0.314

* Update branding to 8.0.411

* Delete src/arcade/eng/common directory

It got accidentally added due to a darc bug, see dotnet/arcade-services#4837

* Delete src/arcade/eng/common directory

---------

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Sean Reeser <[email protected]>
Co-authored-by: Jason Zhai <[email protected]>
Co-authored-by: dotnet-maestro[bot] <42748379+dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Marc Paine <[email protected]>
Co-authored-by: Alexander Köplinger <[email protected]>
Co-authored-by: Michael Yanni <[email protected]>
akoeplinger added a commit to dotnet/templating that referenced this issue May 20, 2025
It got accidentally added due to a darc bug, see dotnet/arcade-services#4837
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants