Skip to content

Updated all download links to just github, can't access download.dre …#2549

Merged
jwillemsen merged 2 commits into
DOCGroup:masterfrom
jwillemsen:jwi-nodownloaddredownload
Jul 9, 2026
Merged

Updated all download links to just github, can't access download.dre …#2549
jwillemsen merged 2 commits into
DOCGroup:masterfrom
jwillemsen:jwi-nodownloaddredownload

Conversation

@jwillemsen

@jwillemsen jwillemsen commented Jul 9, 2026

Copy link
Copy Markdown
Member

…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:

Summary by CodeRabbit

  • Documentation
    • Updated ACE and TAO documentation download links to use the GitHub-hosted repository instead of the Vanderbilt URLs.
    • Refreshed ACE+TAO, CIAO, and DAnCE download tables to remove FTP references and show HTTP-only options for current releases.
    • Updated install and tutorial pages to direct readers to the latest ACE+TAO release guidance and GitHub-based download links.

…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:
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 6ec479d2-9a7e-47b6-9e90-373df6150882

📥 Commits

Reviewing files that changed from the base of the PR and between 1be5b48 and 6b394ac.

📒 Files selected for processing (1)
  • ACE/README
✅ Files skipped from review due to trivial changes (1)
  • ACE/README

Walkthrough

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

Changes

Download link migration

Layer / File(s) Summary
ACE overview links
ACE/README, ACE/Kokyu/docs/Kokyu.html
ACE availability text and Kokyu status link were updated to use HTTP-only wording and the GitHub repository reference.
ACE download tables
ACE/docs/Download.html
ACE+TAO, CIAO, and DAnCE release tables now show only GitHub HTTP download links and no FTP alternatives.
TAO install and download links
TAO/TAO-INSTALL.html, TAO/docs/Security/Download.html, TAO/docs/tutorials/Quoter/index.html
TAO installation text and download links were updated to reference the GitHub repository instead of Vanderbilt URLs.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

  • DOCGroup/ACE_TAO#2377: Both PRs modify the download tables and link targets in ACE/docs/Download.html.
  • DOCGroup/ACE_TAO#2539: Both PRs edit the same ACE release download sections and link presentation in ACE/docs/Download.html.
  • DOCGroup/ACE_TAO#2344: Both PRs update TAO installation/download documentation links in TAO/TAO-INSTALL.html.

Poem

I’m a rabbit with a doc-edit spree,
Hopping from Vanderbilt to GitHub’s tree.
FTP paths fade out of sight,
HTTP links now feel just right.
🐇📚

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the main change: replacing download.dre links with GitHub download links across the docs.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mergify

mergify Bot commented Jul 9, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
ACE/README (1)

5-6: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Stale 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

📥 Commits

Reviewing files that changed from the base of the PR and between a11cc7c and 1be5b48.

📒 Files selected for processing (6)
  • 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

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

    * ACE/README:
@jwillemsen jwillemsen merged commit 07c010d into DOCGroup:master Jul 9, 2026
38 of 39 checks passed
@jwillemsen jwillemsen deleted the jwi-nodownloaddredownload branch July 9, 2026 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant