Update ruff configuration - Githubissues

vacanza / python-holidays

Generate and work with holidays in Python

https://pypi.org/project/holidays

MIT License

1.39k stars 447 forks source link

Update ruff configuration #1819

Closed arkid15r closed 2 weeks ago

arkid15r commented 3 weeks ago

Proposed change

Update ruff configuration:

enforce E5 category, delete E501 suppressions
enforce N category, add N802 suppressions
enforce W category

Type of change

[ ] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[x] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency update (version deprecation/upgrade)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] I've run make pre-commit, it didn't generate any changes
[x] I've run make test, all tests passed locally

sonarcloud[bot] commented 3 weeks ago

Quality Gate passed

Issues
25 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

coveralls commented 3 weeks ago

Pull Request Test Coverage Report for Build 9421834272

Details

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 9393429174:	0.0%
Covered Lines:	11338
Relevant Lines:	11338

💛 - Coveralls