Skip to content

Linting: Remove all trailing whitespaces (C0303) and harmonize some docstrings#2995

Merged
cbentejac merged 3 commits intodevelopfrom
dev/lintingWhitespaces
Feb 12, 2026
Merged

Linting: Remove all trailing whitespaces (C0303) and harmonize some docstrings#2995
cbentejac merged 3 commits intodevelopfrom
dev/lintingWhitespaces

Conversation

@cbentejac
Copy link
Copy Markdown
Contributor

Description

This PR removes all the C0303 linting warnings (trailing-whitespaces) across the project. Additionally it harmonizes some comments and docstrings. The linting commits are added to the list of commits in .git-blame-ignore-revs.

@cbentejac cbentejac added this to the Meshroom 2026.1.0 milestone Feb 9, 2026
@cbentejac cbentejac requested a review from Alxiice February 9, 2026 15:52
@cbentejac cbentejac self-assigned this Feb 9, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.67%. Comparing base (35381b9) to head (366f89a).
⚠️ Report is 17 commits behind head on develop.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #2995   +/-   ##
========================================
  Coverage    81.67%   81.67%           
========================================
  Files           69       69           
  Lines         9144     9144           
========================================
  Hits          7468     7468           
  Misses        1676     1676           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cbentejac cbentejac merged commit aff0c40 into develop Feb 12, 2026
3 of 4 checks passed
@cbentejac cbentejac deleted the dev/lintingWhitespaces branch February 12, 2026 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants