Skip to content

Fix for<inheritdoc/> not respected by IntelliSense #262

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Jun 18, 2025
Merged

Conversation

ChrisPulman
Copy link
Member

What kind of change does this PR introduce?

fixes #260

What is the current behavior?

#260

What is the new behavior?

This pull request introduces updates to the ReactiveUI source generators, primarily adding metadata annotations for generated code and improving code coverage exclusions. These changes enhance the maintainability and clarity of the generated code, particularly for attributes and observable properties.

Updates to attributes:

  • Added [global::System.CodeDom.Compiler.GeneratedCode] annotation to the BindableDerivedListAttribute class to indicate the source generator version used for its creation. (src/ReactiveUI.SourceGenerator.Tests/DERIVEDLIST/BindableDerivedListGeneratorTests.FromReactiveProperties#ReactiveUI.SourceGenerators.BindableDerivedListAttribute.g.verified.cs)
  • Added [global::System.CodeDom.Compiler.GeneratedCode] annotation to the ObservableAsPropertyAttribute class across multiple test files to specify the generator version and enhance traceability. (src/ReactiveUI.SourceGenerator.Tests/OAPH/OAPFromObservableGeneratorTests.*#ReactiveUI.SourceGenerators.ObservableAsPropertyAttribute.g.verified.cs)

Updates to observable properties:

  • Added [global::System.CodeDom.Compiler.GeneratedCode] and [global::System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage] annotations to observable property helpers and their initialization methods in generated classes to improve code clarity and exclude generated code from coverage metrics. (src/ReactiveUI.SourceGenerator.Tests/OAPH/OAPFromObservableGeneratorTests.*#TestNs.TestVM.ObservableAsPropertyFromObservable.g.verified.cs) [1] [2] [3] [4] [5] [6] [7]
  • Enhanced serialization support for observable properties by adding [global::System.Runtime.Serialization.DataMemberAttribute] and [global::System.Text.Json.Serialization.JsonIncludeAttribute] annotations where applicable. (src/ReactiveUI.SourceGenerator.Tests/OAPH/OAPFromObservableGeneratorTests.*#TestNs.TestVM.ObservableAsPropertyFromObservable.g.verified.cs) [1] [2] [3]

These changes collectively refine the generated code, making it more robust, traceable, and easier to integrate with serialization frameworks and testing tools.

What might this PR break?

None expected

Please check if the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Other information:

@ChrisPulman ChrisPulman merged commit 301e012 into main Jun 18, 2025
1 check passed
@ChrisPulman ChrisPulman deleted the CP_FixFor260 branch June 18, 2025 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: <inheritdoc/> is not respected by IntelliSense due to attribute order in generated properties.
2 participants