Skip to content

[generate-type-forwarders] Fix incorrect overrides on methods and properties - #10978

Merged
spouliot merged 1 commit into
dotnet:mainfrom
spouliot:gtf-fix-overrides
Mar 26, 2021
Merged

[generate-type-forwarders] Fix incorrect overrides on methods and properties#10978
spouliot merged 1 commit into
dotnet:mainfrom
spouliot:gtf-fix-overrides

Conversation

@spouliot

Copy link
Copy Markdown
Contributor

Examples

Modified properties:

-public virtual SceneKit.SCNScene Scene { get; set; }
+public override SceneKit.SCNScene Scene { get; set; }

Modified methods:

-public virtual void CancelRequest (Foundation.NSError error)
+public override void CancelRequest (Foundation.NSError error)

…perties

Examples

Modified properties:

```diff
-public virtual SceneKit.SCNScene Scene { get; set; }
+public override SceneKit.SCNScene Scene { get; set; }
```

Modified methods:

```diff
-public virtual void CancelRequest (Foundation.NSError error)
+public override void CancelRequest (Foundation.NSError error)
```
@spouliot spouliot added the not-notes-worthy Ignore for release notes label Mar 26, 2021
@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

✅ Tests passed on Build. ✅

Tests passed on Build.

API diff

✅ API Diff from stable

View API diff

🎉 All 182 tests passed 🎉

Pipeline on Agent XAMBOT-1095

@spouliot
spouliot merged commit fc5c571 into dotnet:main Mar 26, 2021
@spouliot
spouliot deleted the gtf-fix-overrides branch March 26, 2021 13:34
tj-devel709 pushed a commit that referenced this pull request Apr 7, 2021
…perties (#10978)

Examples

Modified properties:

```diff
-public virtual SceneKit.SCNScene Scene { get; set; }
+public override SceneKit.SCNScene Scene { get; set; }
```

Modified methods:

```diff
-public virtual void CancelRequest (Foundation.NSError error)
+public override void CancelRequest (Foundation.NSError error)
```
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.

3 participants