Skip to content

Include regex compiler detail in CRegExp errors - #996

Open
AbhiramVSA wants to merge 1 commit into
ipython:mainfrom
AbhiramVSA:fix/cregexp-error-details
Open

AbhiramVSA wants to merge 1 commit into
ipython:mainfrom
AbhiramVSA:fix/cregexp-error-details

Conversation

@AbhiramVSA

Copy link
Copy Markdown

Config startup logs str(TraitError), but that leaves out why a CRegExp pattern failed to compile.

This keeps the compiler's explanation through the existing info path while preserving TraitError. The regression cases cover a named trait and a nested list.

Fixes #475

🤖🤖

@codecov

codecov Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.22%. Comparing base (a576f24) to head (104482e).

Components Coverage Δ
traitlets 85.64% <100.00%> (+<0.01%) ⬆️
tests 99.07% <100.00%> (+<0.01%) ⬆️

☔ View full report in Codecov by Harness.
📢 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.

This branch has not been deployed

No deployments
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.

CRegExp hides usefull exception when compiling pattern

1 participant