Skip to content

[runtime] Skip early from xamarin_pinvoke_override to avoid a later logging statement. - #12443

Merged
rolfbjarne merged 1 commit into
dotnet:mainfrom
rolfbjarne:runtime-skip-logging
Aug 17, 2021
Merged

[runtime] Skip early from xamarin_pinvoke_override to avoid a later logging statement.#12443
rolfbjarne merged 1 commit into
dotnet:mainfrom
rolfbjarne:runtime-skip-logging

Conversation

@rolfbjarne

Copy link
Copy Markdown
Member

This avoids unnecessary and confusing logging which apparently claims that we
couldn't find a symbol, when we didn't even try (on purpose).

…ogging statement.

This avoids unnecessary and confusing logging which apparently claims that we
couldn't find a symbol, when we didn't even try (on purpose).
@rolfbjarne rolfbjarne added the not-notes-worthy Ignore for release notes label Aug 16, 2021
@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

✅ [PR Build] Tests passed on Build. ✅

Tests passed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

API Diff (from PR only) (no change)
Generator Diff (only version changes)

GitHub pages

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

🎉 All 101 tests passed 🎉

Pipeline on Agent XAMBOT-1101.BigSur'
Merge 01a00a6 into e396948

@rolfbjarne
rolfbjarne merged commit 7f5fcfc into dotnet:main Aug 17, 2021
@rolfbjarne
rolfbjarne deleted the runtime-skip-logging branch August 17, 2021 06:05
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants