Skip to content

Fix config lookup: use 'is None' instead of 'or default', add get_config_flex and html_extra_path support#102

Merged
timkpaine merged 1 commit intopython-project-templates:mainfrom
Point72:tkp/flex
Apr 13, 2026
Merged

Fix config lookup: use 'is None' instead of 'or default', add get_config_flex and html_extra_path support#102
timkpaine merged 1 commit intopython-project-templates:mainfrom
Point72:tkp/flex

Conversation

@timkpaine
Copy link
Copy Markdown
Member

  • Add get_config_flex() to accept both hyphens and underscores in config keys
  • Replace 'or default' pattern with 'is None' check in configuration_args loop to avoid overwriting intentional falsy values (False, [], 0, etc.)
  • Switch use_autoapi, autoapi_ignore, custom_css, custom_js lookups to get_config_flex for flexible key matching
  • Add html_extra_path support in build.py and conf.py.j2, with relative-to- absolute path conversion
  • Add tests for get_config_flex and html_extra_path

…fig_flex and html_extra_path support

- Add get_config_flex() to accept both hyphens and underscores in config keys
- Replace 'or default' pattern with 'is None' check in configuration_args loop
  to avoid overwriting intentional falsy values (False, [], 0, etc.)
- Switch use_autoapi, autoapi_ignore, custom_css, custom_js lookups to
  get_config_flex for flexible key matching
- Add html_extra_path support in build.py and conf.py.j2, with relative-to-
  absolute path conversion
- Add tests for get_config_flex and html_extra_path
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.48%. Comparing base (ddc99c5) to head (db97311).
⚠️ Report is 17 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #102      +/-   ##
==========================================
+ Coverage   78.84%   80.48%   +1.63%     
==========================================
  Files           8        8              
  Lines        1130     1230     +100     
  Branches      134      138       +4     
==========================================
+ Hits          891      990      +99     
  Misses        201      201              
- Partials       38       39       +1     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@timkpaine timkpaine merged commit ca52843 into python-project-templates:main Apr 13, 2026
3 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.

1 participant