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

Build fix - remove unnecessary 'using' #5512

Merged
merged 1 commit into from
Oct 11, 2024

Conversation

SteveSandersonMS
Copy link
Member

@SteveSandersonMS SteveSandersonMS commented Oct 11, 2024

Microsoft Reviewers: Open in CodeFlow

@stephentoub
Copy link
Member

How did this get through CI? Multiple parallel prs?

@eiriktsarpalis
Copy link
Member

It's also being removed here: https://github.com/dotnet/extensions/pull/5510/files

@SteveSandersonMS
Copy link
Member Author

SteveSandersonMS commented Oct 11, 2024

How did this get through CI? Multiple parallel prs?

#5507 may have been in flight at the same time as #5505. But I haven't done enough investigation to prove this is the cause.

It's also being removed here: https://github.com/dotnet/extensions/pull/5510/files

Great, thanks. Let's just merge whichever one of the PRs is ready first, so as to unblock the build.

@SteveSandersonMS SteveSandersonMS merged commit a8e910a into main Oct 11, 2024
6 checks passed
@SteveSandersonMS SteveSandersonMS deleted the stevesa/fix-build-remove-using branch October 11, 2024 16:18
@github-actions github-actions bot locked and limited conversation to collaborators Nov 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants