Skip to content

Fix internal link errors#5634

Open
pipliggins wants to merge 8 commits into
pybamm-team:mainfrom
pipliggins:fix-changelog-path
Open

Fix internal link errors#5634
pipliggins wants to merge 8 commits into
pybamm-team:mainfrom
pipliggins:fix-changelog-path

Conversation

@pipliggins

@pipliggins pipliggins commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Fixes various errors in the url checker for internal links which weren't updated during/after the monorepo merge.

The remaining error in scripts/update_version.py is because no pybamm-v{version} tags (the new naming convention after the monorepo merge) have been generated yet.

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@pipliggins pipliggins changed the title Fix path errors for CHANGELOG Fix internal link errors Jun 29, 2026
@pipliggins pipliggins marked this pull request as ready for review June 29, 2026 20:28
@pipliggins pipliggins requested a review from a team as a code owner June 29, 2026 20:28
@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.09%. Comparing base (8013d49) to head (0515345).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5634      +/-   ##
==========================================
- Coverage   98.19%   98.09%   -0.10%     
==========================================
  Files         339      339              
  Lines       32069    32069              
==========================================
- Hits        31489    31459      -30     
- Misses        580      610      +30     

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

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