Updated all download links to just github, can't access download.dre …#2549
Conversation
…anymore for new uploads
* ACE/Kokyu/docs/Kokyu.html:
* ACE/README:
* ACE/docs/Download.html:
* TAO/TAO-INSTALL.html:
* TAO/docs/Security/Download.html:
* TAO/docs/tutorials/Quoter/index.html:
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
WalkthroughDocumentation across ACE and TAO now points to the GitHub ACE_TAO repository instead of Vanderbilt download URLs. ACE download tables also remove FTP alternatives and keep only HTTP GitHub links. ChangesDownload link migration
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
|
Tick the box to add this pull request to the merge queue (same as
|
There was a problem hiding this comment.
🧹 Nitpick comments (1)
ACE/README (1)
5-6: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueStale FTP reference in introductory text.
Lines 5–6 still state ACE is "available via both anonymous ftp and http," but this PR is removing FTP download links in favor of GitHub. Consider updating this text to reflect the new distribution model.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@ACE/README` around lines 5 - 6, Update the introductory distribution text in the README to remove the outdated anonymous FTP reference and align it with the new GitHub-based download/distribution approach. Locate the opening availability sentence in the README and revise it so it only mentions the currently supported access method, keeping the wording consistent with the rest of the documentation.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@ACE/README`:
- Around line 5-6: Update the introductory distribution text in the README to
remove the outdated anonymous FTP reference and align it with the new
GitHub-based download/distribution approach. Locate the opening availability
sentence in the README and revise it so it only mentions the currently supported
access method, keeping the wording consistent with the rest of the
documentation.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: e01339d7-3ee7-4ef8-9664-b6b0f53e6951
📒 Files selected for processing (6)
ACE/Kokyu/docs/Kokyu.htmlACE/READMEACE/docs/Download.htmlTAO/TAO-INSTALL.htmlTAO/docs/Security/Download.htmlTAO/docs/tutorials/Quoter/index.html
Up to standards ✅🟢 Issues
|
…anymore for new uploads
Summary by CodeRabbit