docs: fix documentation relative links and normalize test file naming#1730
docs: fix documentation relative links and normalize test file naming#1730achahid19 wants to merge 3 commits intohiero-ledger:mainfrom
Conversation
Remove invalid TOC entries in CONTRIBUTING.md, fix relative paths in testing.md, convert Windows paths to Unix format, and rename test files to follow *_test.py naming convention. Signed-off-by: achahid19 <anasks1999@gmail.com>
Signed-off-by: achahid19 <anasks1999@gmail.com>
WalkthroughFixes broken documentation links and normalizes test file path references across docs; removes two outdated Table of Contents entries in CONTRIBUTING.md and adds a changelog entry documenting these documentation updates. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
tech0priyanshu
left a comment
There was a problem hiding this comment.
I checked the relative path, but I’m still getting a 404 for all testing.md
|
Thanks @achahid19 for taking the time and putting in the ef |
|
Hi @tech0priyanshu |
|
Hi @tech0priyanshu @exploreriii, I looked into the issue and here’s what I found: The relative paths I added work correctly locally and when browsing my fork branch fix_docs However, in the Pull Request UI, GitHub doesn’t reliably resolve relative links for PRs coming from a fork, so the links lose their branch context and return 404 errors. Since the referenced files already exist in the repository, would you prefer that I replace these with absolute links pointing to the Let me know what approach you’d like me to follow. |
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #1730 +/- ##
=======================================
Coverage 93.24% 93.24%
=======================================
Files 141 141
Lines 9051 9051
=======================================
Hits 8440 8440
Misses 611 611 🚀 New features to boost your workflow:
|
Description:
Fix broken documentation links and align test file references with repository conventions.
CONTRIBUTING.mdtable of contentstesting.mdtest_*.pyto*_test.pysdk_developersfolderRelated issue(s):
Fixes #1706
Notes for reviewer:
This PR only updates documentation links, path formatting, and test file naming references. No functional code changes.
Checklist