Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Failed] Auto refactor by Resharper CLT #892

Merged
merged 63 commits into from
Nov 17, 2020
Merged

Conversation

kreghek
Copy link
Owner

@kreghek kreghek commented Nov 14, 2020

No description provided.

@kreghek kreghek added code improvement Улучшение метрик кодовой базы release-significant This merge request will log into release notes labels Nov 14, 2020
@kreghek kreghek added this to the 2.4.7 Stability milestone Nov 14, 2020
@kreghek kreghek self-assigned this Nov 14, 2020

using var file = new StreamWriter(filename, append: true);
Copy link
Owner Author

Choose a reason for hiding this comment

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

Save argument names

public int TopCoord { get; set; }

public int BottomCoord { get; set; }
Copy link
Owner Author

Choose a reason for hiding this comment

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

Not sorted

public RegisterServices RegisterServices { get; }

public IGlobe Globe { get; private set; }

Copy link
Owner Author

Choose a reason for hiding this comment

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

Not sorted

@kreghek kreghek removed this from the 2.4.7 Stability milestone Nov 17, 2020
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@kreghek kreghek merged commit 93d96a1 into master Nov 17, 2020
@kreghek kreghek deleted the auto-refactor-resharper branch November 17, 2020 08:10
@kreghek kreghek removed the release-significant This merge request will log into release notes label Nov 17, 2020
@kreghek kreghek changed the title Auto refactor by Resharper CLT [Failed] Auto refactor by Resharper CLT Nov 17, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
code improvement Улучшение метрик кодовой базы
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant