Skip to content

Fix flake8 issue in spellcheck test#20

Merged
christianrauch merged 5 commits into
colcon:mainfrom
MatthijsBurgh:patch-1
Feb 24, 2026
Merged

Fix flake8 issue in spellcheck test#20
christianrauch merged 5 commits into
colcon:mainfrom
MatthijsBurgh:patch-1

Conversation

@MatthijsBurgh

Copy link
Copy Markdown
Contributor

No description provided.

@codecov-commenter

codecov-commenter commented Feb 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 20.71%. Comparing base (ab68d36) to head (27cb66a).

Files with missing lines Patch % Lines
colcon_meson/build.py 0.00% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
- Coverage   21.22%   20.71%   -0.51%     
==========================================
  Files           2        2              
  Lines         245      251       +6     
  Branches       49       51       +2     
==========================================
  Hits           52       52              
- Misses        186      192       +6     
  Partials        7        7              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread test/test_spell_check.py
@christianrauch christianrauch self-requested a review February 23, 2026 22:47
Comment thread colcon_meson/build.py
@christianrauch

Copy link
Copy Markdown
Contributor

Thanks a lot for also fixing the API breakage with newer Meson versions.

@christianrauch christianrauch merged commit 40d39e3 into colcon:main Feb 24, 2026
47 checks passed
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.

3 participants