Skip to content

Conversation

@NickKhalow
Copy link
Contributor

@NickKhalow NickKhalow commented Nov 17, 2025

Pull Request Description

What does this PR change?

PR applies linting rules to CI that are specified in the .editorconfig. This PR removes some deprecated code warnings as well and other linting errors.

CI commands are based on resharper CLI: https://www.jetbrains.com/help/rider/InspectCode.html

At first run it showed an enourmous amount of warnings, we should define what is acceptable and what is not. Otherwise the changes that we should apply won't fit under one PR. https://drive.google.com/file/d/1UJYI1HVyvQTQPV_mF_ggtdQmEvNbYof5/view?usp=sharing

Test Instructions

Test Steps

  1. Sanity check

Quality Checklist

  • [+] Changes have been tested locally

Code Review Reference

Please review our Code Review Standards before submitting.

@NickKhalow NickKhalow self-assigned this Nov 17, 2025
@NickKhalow NickKhalow requested review from a team as code owners November 17, 2025 13:49
@m3taphysics m3taphysics added force-build Used to trigger a build on draft PR and removed force-build Used to trigger a build on draft PR labels Nov 18, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 18, 2025

badge

Build failed! Check the logs to see what went wrong

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.

5 participants