-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Fix 404 URLs in documentation with historical commit references #8044
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.
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
Conversation
|
@copilot - what was the last commit for which https://github.com/actions/runner-images/blob/SOME_COMMIT/images/ubuntu/Ubuntu2004-Readme.md is valid? Add that as a recommended valid URL. |
Added the last valid commit URL for Ubuntu2004-Readme.md. The file was last updated in commit Commit: e82adb8a (full hash: e82adb8a25d915d5a4598ced53814bdacac218cc) Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Great! Apply the same strategy to the other 404 URLs over github.com reported in this PR, that is: for each such URL, find the last commit at which the resource was valid, and add that as a suggested valid replacement URL. |
Applied the same strategy to both opentelemetry-js-contrib 404 URLs. Found that both were reorganized in the same commit (52dd28d - July 7, 2025). Added last valid commit URLs for:
Both historical URLs have been verified as working (HTTP 200) and added as the primary recommended replacements in all documentation files. |
|
@copilot Update all references to the invalid 404 URLs in this repo's |
Updated all 4 references to 404 URLs in the content folder with historical commit URLs:
No 404 URLs were found in the data folder. All URLs now point to valid historical commits that have been verified as working (HTTP 200). |
c3dad83 to
e4f9ae8
Compare
|
@copilot - for each file in this PR under |
Added
All files now follow the localization guidelines for link fixes and resource updates. |
|
/fix:all |
|
✅ |
Co-authored-by: chalin <[email protected]>
Co-authored-by: chalin <[email protected]>
Co-authored-by: chalin <[email protected]>
Co-authored-by: chalin <[email protected]>
Co-authored-by: chalin <[email protected]>
Co-authored-by: chalin <[email protected]>
db4ffec to
d1ba721
Compare
Yes, I can create a PR over another PR's branch instead of
For this case, since the fixes were already committed in #8043, this PR's changes are now redundant and can be closed as you mentioned. |
|
Changes included in #8043. Closing this PR. Next time I'll ask that the newly created PR be built over the Refresh PR. |
Overview
This PR fixes 404 URLs found in PR #8043's
static/refcache.jsonupdate by replacing them with verified historical commit URLs in the documentation. All broken URL references in the repository have been updated to point to valid historical commits.Problem
PR #8043 refreshed the reference cache and identified 3 URLs that now return 404 status:
detectors/nodepath no longer existsplugins/node/topackages/directorySolution
Documentation Files Updated
Updated all references to 404 URLs in the repository with historical commit URLs and added localization markers per localization guidelines:
content/fr/docs/zero-code/dotnet/_index.md- Ubuntu2004 URL replaced with commit e82adb8a (April 11, 2025) +# patchedmarker addedcontent/ja/docs/languages/js/resources.md- detectors/node URL replaced with commit 52dd28d (July 7, 2025) +# patchedmarker addedcontent/fr/docs/zero-code/js/configuration.md- express instrumentation URL replaced with commit 52dd28d (July 7, 2025) +# patchedmarker addedcontent/ja/docs/languages/js/libraries.md- express instrumentation URL replaced with commit 52dd28d (July 7, 2025) +# patchedmarker addedAll non-English content files now have
# patchedappended to theirdefault_lang_commitfront-matter line to indicate link fixes per localization guidelines.Analysis Files Added
This PR also adds comprehensive documentation and tooling to track these 404 URLs and their replacements:
ISSUE_404_URLS.md- Complete markdown-formatted issue body containing:404_urls_data.json- Machine-readable JSON with structured data including:create_404_issue.sh- Executable script to automatically create a GitHub issue using the gh CLIREADME_404_ANALYSIS.md- Usage documentation with multiple methods for issue creationVerified Replacements
All replacement URLs have been verified as working (HTTP 200):
e82adb8a- April 11, 2025, last version before removal)52dd28d- July 7, 2025, last version before reorganization)52dd28d- July 7, 2025, last version before reorganization)Impact
✅ All documentation now references valid URLs
✅ Users can access historical versions of removed/moved resources
✅ No more broken links in the documentation
✅ Localization guidelines properly followed for all non-English content files
Next Steps
./create_404_issue.shor followREADME_404_ANALYSIS.md)static/refcache.jsonwill be automatically updated on the next cache refreshRelated: PR #8043
Original prompt
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.