8000 fix: Update Resource Path Handling for Configuration Files by eli64s · Pull Request #143 · eli64s/readme-ai · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: Update Resource Path Handling for Configuration Files #143

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Mar 18, 2025
Merged

fix: Update Resource Path Handling for Configuration Files #143

merged 3 commits into from
Mar 18, 2025

Conversation

eli64s
Copy link
Owner
@eli64s eli64s commented Mar 18, 2025 •
edited
Loading

Problem

ReadmeAI fails during installation and execution with:

FileNotFoundError: No such file or directory: 'readmeai/config/settings/templates/headers.toml'

Solution

Changes

  • Updated pyproject.toml to explicitly include settings.template submodule files.
  • Fixed HeaderTemplate class to use proper resource loading.
  • Enhanced "back to top" navigation with shield badge.

Testing

Verified fix through:

  • Test deployments to TestPyPI, see here
  • Installation via pip with dependencies
  • Successful execution with example repositories

@eli64s eli64s added the bug Something isn't working label Mar 18, 2025
@eli64s eli64s self-assigned this Mar 18, 2025
@eli64s eli64s mentioned this pull request Mar 18, 2025
Closed
@eli64s eli64s merged commit d5283a7 into main Mar 18, 2025
5 checks passed
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.21%. Comparing base (1ba63ac) to head (6dfd61d).
Report is 1 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #143      +/-   ##
==========================================
+ Coverage   90.19%   90.21%   +0.01%     
==========================================
  Files          54       54              
  Lines        2692     2697       +5     
  Branches      262      262              
==========================================
+ Hits         2428     2433       +5     
  Misses        264      264

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

@eli64s eli64s deleted the fix/142-missing-headers-toml branch March 18, 2025 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@eli64s eli64s

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Error after installation: No such file or directory: 'readmeai/config/settings/templates/headers.toml'
2 participants
@eli64s @codecov-commenter
0