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

Fix CodeQL issues #285

Merged
merged 1 commit into from Feb 27, 2023
Merged

Fix CodeQL issues #285

merged 1 commit into from Feb 27, 2023

Conversation

sungam3r
Copy link
Member

No description provided.

@github-actions github-actions bot added the tests Pull request that adds new or changes existing tests label Feb 27, 2023
@@ -139,12 +139,12 @@
{
public class PropertyReadWrite
{
public string Value { get { return string.Empty; } set { } }
public string Value { get => string.Empty; set { } }

Check warning

Code scanning / CodeQL

Property value is not used when setting a property

Value ignored when setting property.
@@ -12,8 +12,7 @@
{
get
{
if (_nullableContextProviders == null)
_nullableContextProviders = new Stack<ICustomAttributeProvider>();
_nullableContextProviders ??= new Stack<ICustomAttributeProvider>();

Check notice

Code scanning / CodeQL

Static field written by instance method

Write to static field from instance method or constructor.
@sungam3r sungam3r added the code style Pull request that changes or applies code style rules label Feb 27, 2023
@sungam3r sungam3r merged commit bbe8199 into master Feb 27, 2023
@sungam3r sungam3r deleted the codeql branch February 27, 2023 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code style Pull request that changes or applies code style rules tests Pull request that adds new or changes existing tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant