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

refactor: convert string interpolations with missing arguments to literals #266

Merged
merged 3 commits into from
Aug 2, 2024

Conversation

deepsource-autofix[bot]
Copy link
Contributor

String interpolation requires that you specify the string that is to be interpolated and the required arguments. Failing to specify the required arguments may result in an ill-formatted string. Because it is possible that such strings can critically alter the behaviour and control-flow of the program, it is suggested that you fix it as soon as possible.

…erals

String interpolation requires that you specify the string that is to be interpolated and the required arguments. Failing to specify the required arguments may result in an ill-formatted string. Because it is possible that such strings can critically alter the behaviour and control-flow of the program, it is suggested that you fix it as soon as possible.
This commit fixes the style issues introduced in 0fe2afc according to the output
from dotnet-format.

Details: #266
Copy link
Contributor

deepsource-io bot commented Aug 2, 2024

Here's the code health analysis summary for commits 8da59b3..eb603aa. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Docker LogoDocker✅ SuccessView Check ↗
DeepSource C# LogoC#✅ Success
🎯 4 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@peterliapin peterliapin merged commit 7f324a5 into develop Aug 2, 2024
0 of 3 checks passed
@peterliapin peterliapin deleted the deepsource-autofix-ad5425d4 branch August 2, 2024 08:59
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.

1 participant