-
Notifications
You must be signed in to change notification settings - Fork 77
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
Add GetEffectiveRuleDetailsResponse & EffectiveRuleDetailsDto #5197
Add GetEffectiveRuleDetailsResponse & EffectiveRuleDetailsDto #5197
Conversation
@@ -13,4 +13,8 @@ | |||
<ProjectReference Include="..\TestInfrastructure\TestInfrastructure.csproj" /> | |||
</ItemGroup> | |||
|
|||
<ItemGroup> | |||
<PackageReference Update="FluentAssertions" Version="6.12.0" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I updated it for only one project, but maybe I should have done it for https://github.com/SonarSource/sonarlint-visualstudio/blob/86d1c450a50aa31bb3390f9a124390f01fa73337/src/SonarLint.Test.props
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, you should do it there instead. We don't want to maintain in many places. Also why did you need to change version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I needed it for a feature that was not available in the old one
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please move the new version of Fluent Assertions to SonarLint.Test.props
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
a4613c8
into
feature/sloop-rule-meta-data
Fixes #5193