New issue

Jump to bottom

Code Style Housekeeping #462

Merged

axunonb merged 1 commit into main from pr/code-housekeeping

Jan 16, 2025

+41 −45

Member

axunonb commented Jan 16, 2025

Replaced Any() with Count != 0 for better performance.
Updated parameter types in TimeTextInfo.cs and EvaluatorTests.cs.
Simplified assertion syntax in multiple test files.
Changed Enum.Parse to Enum.TryParse in Address.cs.
Converted instance methods to static in StringSource.cs.

axunonb force-pushed the pr/code-housekeeping branch from 863f146 to b7df185 Compare

January 16, 2025 22:19


          Code Style Housekeeping

217fe4e

- Replaced Any() with Count != 0 for better performance.
- Updated parameter types in TimeTextInfo.cs and EvaluatorTests.cs.
- Simplified assertion syntax in multiple test files.
- Changed Enum.Parse to Enum.TryParse in Address.cs.
- Converted instance methods to static in StringSource.cs.

axunonb force-pushed the pr/code-housekeeping branch from b7df185 to 217fe4e Compare

January 16, 2025 22:21

sonarqubecloud bot commented Jan 16, 2025

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
87.5% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

axunonb merged commit e74de13 into main

5 of 7 checks passed

axunonb deleted the pr/code-housekeeping branch

January 16, 2025 22:26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet