Skip to content

[src] Add a global namespace using for a couple of namespaces. - #14356

Merged
rolfbjarne merged 2 commits into
dotnet:mainfrom
rolfbjarne:dotnet-global-namespaces
Mar 10, 2022
Merged

[src] Add a global namespace using for a couple of namespaces.#14356
rolfbjarne merged 2 commits into
dotnet:mainfrom
rolfbjarne:dotnet-global-namespaces

Conversation

@rolfbjarne

Copy link
Copy Markdown
Member

Also remove all the 'using System.Runtime.Versioning;' statements everywhere.

@rolfbjarne rolfbjarne added not-notes-worthy Ignore for release notes run-dotnet-tests Run all the .NET tests labels Mar 9, 2022
@rolfbjarne

Copy link
Copy Markdown
Member Author

@chamons this might make #14287 somewhat smaller, it goes in first (or I can wait until you've merged #14287).

@chamons

chamons commented Mar 9, 2022

Copy link
Copy Markdown
Contributor

If you get yours green before I finish fixing the last test and writing one last one (very likely) go ahead. It should be a trivial resolve.

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff
View dotnet API diff
View dotnet legacy API diff
View dotnet iOS-MacCatalayst API diff

API Current PR diff

ℹ️ API Diff (from PR only) (please review changes)

View API diff
View dotnet API diff
View dotnet legacy API diff
View dotnet iOS-MacCatalayst API diff
  • ⚠️ Generator diff comments have not been provided.

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Test results

1 tests failed, 145 tests passed.

Failed tests

  • link all/tvOS - simulator/Release: Failed

Pipeline on Agent XAMBOT-1017.Monterey'
Merge 4481f8d into 9fe2a5c

@rolfbjarne

Copy link
Copy Markdown
Member Author

Test failure is unrelated (https://github.com/xamarin/maccore/issues/1083).

@rolfbjarne
rolfbjarne merged commit 9e5113a into dotnet:main Mar 10, 2022
@rolfbjarne
rolfbjarne deleted the dotnet-global-namespaces branch March 10, 2022 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

not-notes-worthy Ignore for release notes run-dotnet-tests Run all the .NET tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants