Skip to content

Conversation

@StanleySweet
Copy link

@StanleySweet StanleySweet force-pushed the add-is-instead-of-as-rule-reference branch from b029251 to 330c61d Compare May 21, 2025 08:36
Copy link
Member

@dedece35 dedece35 left a comment

Choose a reason for hiding this comment

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

check Definition Of Done : please add next updates :

  • update CHANGELOG.md
  • update RULS.md

@MP-Aubay MP-Aubay changed the title Add documentation for the GCI95 rule GCI95 [Team CSHARP][2025] - Add documentation May 21, 2025
@StanleySweet StanleySweet force-pushed the add-is-instead-of-as-rule-reference branch 2 times, most recently from f831283 to 4a01263 Compare May 21, 2025 09:51
{
Console.WriteLine(anotherObj.Value);
}
----
Copy link
Member

Choose a reason for hiding this comment

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

Could you please add some references to prove this rule ? web pages ou local performance tests ?

Copy link
Author

Choose a reason for hiding this comment

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

Sure. I've added links to the relevant documentation in the rules.json as well as additional notes in this document. The rule was provided from https://github.com/green-code-initiative/creedengo-csharp/blob/main/Rules%20to%20work%20on.md

@StanleySweet StanleySweet force-pushed the add-is-instead-of-as-rule-reference branch 2 times, most recently from 8883414 to da3e535 Compare May 21, 2025 12:43
@StanleySweet StanleySweet force-pushed the add-is-instead-of-as-rule-reference branch from da3e535 to e14ddd6 Compare May 21, 2025 12:48
@github-actions
Copy link
Contributor

This PR has been automatically marked as stale because it has no activity for 60 days.
Please add a comment if you want to keep the issue open. Thank you for your contributions!

@github-actions github-actions bot added the stale label Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants