Skip to content

Conversation

@RobertoRoos
Copy link
Contributor

@RobertoRoos RobertoRoos commented Nov 25, 2025

This changes the ArgumentParser options. This seemed easier than parsing out all ANSI color codes instead.

Fixes

Should first merge:

After #79 we can test coloring in the pipelines.

@codecov
Copy link

codecov bot commented Nov 25, 2025

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@2a2a202). Learn more about missing BASE report.

Files with missing lines Patch % Lines
sphinxarg/ext.py 0.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #74   +/-   ##
=======================================
  Coverage        ?   46.86%           
=======================================
  Files           ?       12           
  Lines           ?      958           
  Branches        ?      205           
=======================================
  Hits            ?      449           
  Misses          ?      468           
  Partials        ?       41           
Flag Coverage Δ
unittests 46.86% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@RobertoRoos RobertoRoos changed the title Added default feature to disable ArgumentParser coloring (fixing #72) Added default feature to disable ArgumentParser coloring Nov 25, 2025
@RobertoRoos RobertoRoos force-pushed the bugfix/colors branch 2 times, most recently from 87de191 to aacb372 Compare November 28, 2025 12:19
@ashb ashb merged commit cf1fbb7 into sphinx-doc:main Dec 1, 2025
14 of 15 checks passed
@RobertoRoos RobertoRoos deleted the bugfix/colors branch December 2, 2025 11:31
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.

2 participants