Skip to content
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

Update Shim Layer: Add wrappers for C# 12 Operations #9365

Merged
merged 1 commit into from
May 30, 2024

Conversation

zsolt-kolbay-sonarsource
Copy link
Contributor

@zsolt-kolbay-sonarsource zsolt-kolbay-sonarsource commented May 30, 2024

Add wrappers for ICollectionExpressionOperation and ISpreadOperation.

Done by copying the OperationInterfaces.xml from the Roslyn repo and removing the nullable reference types with find/replace.

Most of the changes are formatting, the only real changes are at the bottom of the file (the two new interfaces).

Copy link

sonarcloud bot commented May 30, 2024

Quality Gate Passed Quality Gate passed for 'Sonar .NET Java Plugin'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link

sonarcloud bot commented May 30, 2024

Quality Gate Passed Quality Gate passed for 'SonarAnalyzer for .NET'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@@ -3387,4 +3402,70 @@
</Comments>
</Property>
</Node>
</Tree>
<Node Name="ICollectionExpressionOperation" Base="IOperation" HasType="true">
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only non-formatting change in the file starts here.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

? were removed from the signatures on purpose

@zsolt-kolbay-sonarsource zsolt-kolbay-sonarsource marked this pull request as ready for review May 30, 2024 12:57
@zsolt-kolbay-sonarsource zsolt-kolbay-sonarsource added this to the 9.26 milestone May 30, 2024
@github-actions github-actions bot moved this from In progress to Review in progress in Best Kanban May 30, 2024
Copy link
Contributor

@pavel-mikula-sonarsource pavel-mikula-sonarsource left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions github-actions bot moved this from Review in progress to Review approved in Best Kanban May 30, 2024
@pavel-mikula-sonarsource pavel-mikula-sonarsource merged commit 51ae665 into master May 30, 2024
34 checks passed
Best Kanban automation moved this from Review approved to Validate Peach May 30, 2024
@pavel-mikula-sonarsource pavel-mikula-sonarsource deleted the Zsolt/update-IOperation-wrappers branch May 30, 2024 14:43
@zsolt-kolbay-sonarsource zsolt-kolbay-sonarsource moved this from Validate Peach to Done in Best Kanban May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Best Kanban
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants