[Rgen] Complete the BintingTypeAttribute analyzer. #21946

mandel-macaque · 2025-01-12T01:21:04Z

The analyzer now checks the following:

All useages of bindingTypeAttribute have to happen on partial classes.
BidingType can only be used on classes.
BindingType can only be used on static classes.
BindingType can only be used on interfaces.
BindingType can only be used on classes.
BindingType can onlye be used on enums.

Because we have not yet released any of the errors, we have sorted them to be the first ones from 0001-0006.

The analyzer now checks the following: * All useages of bindingTypeAttribute<T> have to happen on partial classes. * BidingType<Class> can only be used on classes. * BindingType<Category> can only be used on static classes. * BindingType<Protocol> can only be used on interfaces. * BindingType can onlye be used on enums. Because we have not yet released any of the errors, we have sorted them to be the first ones from 0001-0006.

Copilot

Copilot reviewed 5 out of 10 changed files in this pull request and generated 2 comments.

Files not reviewed (5)

src/rgen/Microsoft.Macios.Bindings.Analyzer/Resources.Designer.cs: Language not supported
src/rgen/Microsoft.Macios.Bindings.Analyzer/Resources.resx: Language not supported
src/rgen/Microsoft.Macios.Bindings.Analyzer/AnalyzerReleases.Unshipped.md: Evaluated as low risk
src/rgen/Microsoft.Macios.Bindings.Analyzer/BindingTypeSemanticAnalyzer.cs: Evaluated as low risk
src/rgen/Microsoft.Macios.Bindings.Analyzer/Extensions/IBindingTypeAnalyzerExtensions.cs: Evaluated as low risk

src/rgen/Microsoft.Macios.Bindings.Analyzer/Extensions/BaseTypeDeclarationSyntaxExtensions.cs

Copilot · 2025-01-12T01:22:51Z

tests/rgen/Microsoft.Macios.Bindings.Analyzer.Tests/BindingTypeSemanticAnalyzerTests.cs

+			yield return [
+				nonPartialClassBinding,
+				BindingTypeSemanticAnalyzer.RBI0001.Id,
+				"The binding type 'Test.Examples' must declared as a partial class"


The error message has a typo: 'The binding type 'Test.Examples' must declared as a partial class' should be 'The binding type 'Test.Examples' must be declared as a partial class'.

Suggested change

"The binding type 'Test.Examples' must declared as a partial class"

"The binding type 'Test.Examples' must be declared as a partial class"

github-actions · 2025-01-12T01:25:05Z

⚠️ Your code has been reformatted. ⚠️

If this is not desired, add the actions-disable-autoformat label, and revert the reformatting commit.

If files unrelated to your change were modified, try reverting the reformatting commit + merging with the target branch (and push those changes).

mandel-macaque · 2025-01-12T12:13:35Z

/azp run xamarin-macios-pr

…eDeclarationSyntaxExtensions.cs Co-authored-by: Copilot <[email protected]>

azure-pipelines · 2025-01-12T12:13:51Z

Azure Pipelines successfully started running 1 pipeline(s).

github-actions · 2025-01-12T21:45:05Z

⚠️ Your code has been reformatted. ⚠️

If this is not desired, add the actions-disable-autoformat label, and revert the reformatting commit.

If files unrelated to your change were modified, try reverting the reformatting commit + merging with the target branch (and push those changes).

vs-mobiletools-engineering-service2 · 2025-01-12T22:35:40Z

✅ [PR Build] Build passed (Build packages) ✅

Pipeline on Agent
Hash: 3293eaa54f5f590558088799b8bacfa64c9cf75a [PR build]

vs-mobiletools-engineering-service2 · 2025-01-12T22:39:50Z

✅ [PR Build] Build passed (Detect API changes) ✅

Pipeline on Agent
Hash: aac0e4cd5c3ee92e849cccb4bf109f0eded5432f [PR build]

vs-mobiletools-engineering-service2 · 2025-01-12T23:11:34Z

✅ [PR Build] Build passed (Build macOS tests) ✅

Pipeline on Agent
Hash: 3293eaa54f5f590558088799b8bacfa64c9cf75a [PR build]

vs-mobiletools-engineering-service2 · 2025-01-12T23:18:22Z

💻 [CI Build] Windows Integration Tests passed 💻

✅ All Windows Integration Tests passed.

Pipeline on Agent
Hash: 3293eaa54f5f590558088799b8bacfa64c9cf75a [PR build]

vs-mobiletools-engineering-service2 · 2025-01-12T23:23:49Z

💻 [PR Build] Tests on macOS M1 - Mac Monterey (12) passed 💻

✅ All tests on macOS M1 - Mac Monterey (12) passed.

Pipeline on Agent
Hash: 3293eaa54f5f590558088799b8bacfa64c9cf75a [PR build]

vs-mobiletools-engineering-service2 · 2025-01-12T23:24:45Z

💻 [PR Build] Tests on macOS X64 - Mac Sonoma (14) passed 💻

✅ All tests on macOS X64 - Mac Sonoma (14) passed.

Pipeline on Agent
Hash: 3293eaa54f5f590558088799b8bacfa64c9cf75a [PR build]

vs-mobiletools-engineering-service2 · 2025-01-12T23:25:35Z

💻 [PR Build] Tests on macOS arm64 - Mac Sequoia (15) passed 💻

✅ All tests on macOS arm64 - Mac Sequoia (15) passed.

Pipeline on Agent
Hash: 3293eaa54f5f590558088799b8bacfa64c9cf75a [PR build]

vs-mobiletools-engineering-service2 · 2025-01-12T23:25:42Z

💻 [PR Build] Tests on macOS M1 - Mac Ventura (13) passed 💻

✅ All tests on macOS M1 - Mac Ventura (13) passed.

Pipeline on Agent
Hash: 3293eaa54f5f590558088799b8bacfa64c9cf75a [PR build]

vs-mobiletools-engineering-service2 · 2025-01-12T23:43:17Z

✅ API diff for current PR / commit

.NET ( No breaking changes )

iOS: vsdrops gist ( No breaking changes )
tvOS: vsdrops gist ( No breaking changes )
MacCatalyst: vsdrops gist ( No breaking changes )
macOS: vsdrops gist ( No breaking changes )

❗ API diff vs stable (Breaking changes)

.NET ( ❗ Breaking changes ❗ )

iOS: vsdrops gist ( ❗ Breaking changes ❗ )
tvOS: vsdrops gist ( ❗ Breaking changes ❗ )
MacCatalyst: vsdrops gist ( ❗ Breaking changes ❗ )
macOS: vsdrops gist ( ❗ Breaking changes ❗ )

ℹ️ Generator diff

Generator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes)

Pipeline on Agent
Hash: aac0e4cd5c3ee92e849cccb4bf109f0eded5432f [PR build]

vs-mobiletools-engineering-service2 · 2025-01-13T01:31:18Z

🚀 [CI Build] Test results 🚀

Test results

✅ All tests passed on VSTS: test results.

🎉 All 107 tests passed 🎉

Tests counts

✅ cecil: All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (iOS): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (MacCatalyst): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (macOS): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (Multiple platforms): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (tvOS): All 1 tests passed. Html Report (VSDrops) Download
✅ framework: All 2 tests passed. Html Report (VSDrops) Download
✅ fsharp: All 4 tests passed. Html Report (VSDrops) Download
✅ generator: All 5 tests passed. Html Report (VSDrops) Download
✅ interdependent-binding-projects: All 4 tests passed. Html Report (VSDrops) Download
✅ introspection: All 4 tests passed. Html Report (VSDrops) Download
✅ linker: All 40 tests passed. Html Report (VSDrops) Download
✅ monotouch (iOS): All 8 tests passed. Html Report (VSDrops) Download
✅ monotouch (MacCatalyst): All 10 tests passed. Html Report (VSDrops) Download
✅ monotouch (macOS): All 9 tests passed. Html Report (VSDrops) Download
✅ monotouch (tvOS): All 8 tests passed. Html Report (VSDrops) Download
✅ msbuild: All 2 tests passed. Html Report (VSDrops) Download
✅ xcframework: All 4 tests passed. Html Report (VSDrops) Download
✅ xtro: All 1 tests passed. Html Report (VSDrops) Download

Pipeline on Agent
Hash: 3293eaa54f5f590558088799b8bacfa64c9cf75a [PR build]

mandel-macaque requested a review from Copilot January 12, 2025 01:21

Copilot AI reviewed Jan 12, 2025

View reviewed changes

Auto-format source code

53cbe4a

This comment has been minimized.

Sign in to view

Update src/rgen/Microsoft.Macios.Bindings.Analyzer/Extensions/BaseTyp…

4d358e8

…eDeclarationSyntaxExtensions.cs Co-authored-by: Copilot <[email protected]>

Fix small typo found by copilot.

2f94826

This comment has been minimized.

Sign in to view

mandel-macaque and others added 2 commits January 12, 2025 16:40

Take into account strong dictionaries.

aac0e4c

Auto-format source code

3293eaa

This comment has been minimized.

Sign in to view

	"The binding type 'Test.Examples' must declared as a partial class"
	"The binding type 'Test.Examples' must be declared as a partial class"

[Rgen] Complete the BintingTypeAttribute analyzer. #21946

Are you sure you want to change the base?

[Rgen] Complete the BintingTypeAttribute analyzer. #21946

Conversation

mandel-macaque commented Jan 12, 2025 • edited Loading

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot AI Jan 12, 2025

Choose a reason for hiding this comment

github-actions bot commented Jan 12, 2025

⚠️ Your code has been reformatted. ⚠️

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

mandel-macaque commented Jan 12, 2025

azure-pipelines bot commented Jan 12, 2025

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

github-actions bot commented Jan 12, 2025

⚠️ Your code has been reformatted. ⚠️

This comment has been minimized.

This comment has been minimized.

vs-mobiletools-engineering-service2 commented Jan 12, 2025

✅ [PR Build] Build passed (Build packages) ✅

vs-mobiletools-engineering-service2 commented Jan 12, 2025

✅ [PR Build] Build passed (Detect API changes) ✅

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

vs-mobiletools-engineering-service2 commented Jan 12, 2025

✅ [PR Build] Build passed (Build macOS tests) ✅

This comment has been minimized.

vs-mobiletools-engineering-service2 commented Jan 12, 2025

💻 [CI Build] Windows Integration Tests passed 💻

vs-mobiletools-engineering-service2 commented Jan 12, 2025

💻 [PR Build] Tests on macOS M1 - Mac Monterey (12) passed 💻

vs-mobiletools-engineering-service2 commented Jan 12, 2025

💻 [PR Build] Tests on macOS X64 - Mac Sonoma (14) passed 💻

vs-mobiletools-engineering-service2 commented Jan 12, 2025

💻 [PR Build] Tests on macOS arm64 - Mac Sequoia (15) passed 💻

vs-mobiletools-engineering-service2 commented Jan 12, 2025

💻 [PR Build] Tests on macOS M1 - Mac Ventura (13) passed 💻

vs-mobiletools-engineering-service2 commented Jan 12, 2025

✅ API diff for current PR / commit

❗ API diff vs stable (Breaking changes)

ℹ️ Generator diff

This comment has been minimized.

vs-mobiletools-engineering-service2 commented Jan 13, 2025

🚀 [CI Build] Test results 🚀

Test results

Tests counts

mandel-macaque commented Jan 12, 2025 •

edited

Loading