Skip to content

Remove unnecessary explicit override - #6664

Open
Trung Nguyen (trungnt2910) wants to merge 1 commit into
microsoft:mainfrom
trungnt2910:patch-2
Open

Trung Nguyen (trungnt2910) wants to merge 1 commit into
microsoft:mainfrom
trungnt2910:patch-2

Conversation

@trungnt2910

Copy link
Copy Markdown

Since IResourceLinkBuilder is the only base of PriSectionBuilder that defines GetSchema(), the explicit qualifier is redundant.

Removing this MSVC-specific feature also allows MRTCore to be built with Clang and MinGW.

A microsoft employee must use /azp run to validate using the pipelines below.

WARNING:
Comments made by azure-pipelines bot maybe inaccurate.
Please see pipeline link to verify that the build is being ran.

For status checks on the main branch, please use TransportPackage-Foundation-PR
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=81063&_a=summary)
and run the build against your PR branch with the default parameters.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@guimafelipe

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
1 pipeline(s) were filtered out due to trigger conditions.

@microsoft-github-policy-service

Copy link
Copy Markdown
Contributor

This PR has been open for 30 days. Are you still interested in merging this PR? If so, please make sure your branch is up to date with the latest main by merging main into your branch or rebasing onto main.

Since `IResourceLinkBuilder` is the only base of `PriSectionBuilder`
that defines `GetSchema()`, the explicit qualifier is redundant.

Removing this MSVC-specific feature also allows MRTCore to be
built with Clang and MinGW.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants