Skip to content

chore: remove www.accomplish.ai brand links from README#980

Merged
DanielScharfsteinGit merged 1 commit into
mainfrom
cleanup/remove-accomplish-ai-brand-links
May 12, 2026
Merged

chore: remove www.accomplish.ai brand links from README#980
DanielScharfsteinGit merged 1 commit into
mainfrom
cleanup/remove-accomplish-ai-brand-links

Conversation

@DanielScharfsteinGit
Copy link
Copy Markdown
Contributor

@DanielScharfsteinGit DanielScharfsteinGit commented May 12, 2026

Jira

TBD — Slack request from Guy (2026-05-12 10:23 IDT): "Can someone pick a small task of removing the links from GitHub to accomplish.ai? We have them in the description and readme (and maybe elsewhere). It's important for reducing confusion and increase hygiene."

Summary

Removes 4 marketing/brand links to www.accomplish.ai from the public accomplish repo README. Functional downloads.accomplish.ai binary URLs are preserved.

Why

Per Guy's request — hygiene + reduce confusion. The links to the marketing site (website, blog, "Built by Accomplish") no longer reflect where the project lives; the README's job is to point at the GitHub-hosted source + releases.

What Changed

README.md:

  • Removed Accomplish website and Accomplish blog from the top download bar.
  • Removed Accomplish website and Accomplish blog from the footer link row.
  • Removed Built by [Accomplish](https://www.accomplish.ai) trailing the MIT License line.
  • Preserved the 12 functional downloads.accomplish.ai/downloads/0.5.17/... binary URLs and the releases / Issues / Twitter links.

Net diff: 1 file changed, 2 insertions(+), 6 deletions(-).

Follow-up needed (repo-admin only — not in this PR)

The GitHub repo's Homepage URL field is currently set to https://www.accomplish.ai. That field is part of repo settings, not the codebase, so it requires a separate admin action:

gh api -X PATCH repos/accomplish-ai/accomplish -f homepage=''

Flagging here so it doesn't get missed.

Open question for reviewer

If Guy meant to remove all accomplish.ai references (including downloads.accomplish.ai/... and connectors.accomplish.ai), say so — I'll open a follow-on PR with the broader cut. My read of "links from GitHub to accomplish.ai" was marketing-site links specifically, leaving functional/operational URLs in place.

Validation

  • grep -nE "https?://(www\.)?accomplish\.ai" README.md returns zero hits after the change.
  • grep -c "downloads.accomplish.ai" README.md returns 13 (12 binary URLs + 1 in code-block context), confirming functional URLs preserved.
  • README renders correctly locally (markdown lint clean).

Summary by CodeRabbit

  • Documentation
    • Simplified external links and references in documentation by consolidating to key resources (Accomplish releases, Issues, Twitter) and streamlined footer information.

Review Change Stack

Removes 4 marketing links to www.accomplish.ai (header website/blog,
footer website/blog, "Built by Accomplish") per Guy's hygiene request
in #accomplish_app_sprints.

Functional downloads.accomplish.ai binary URLs are preserved (12 total).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d5faeca9-465c-4517-a67a-097e22cdbf60

📥 Commits

Reviewing files that changed from the base of the PR and between 20e57da and a419eef.

📒 Files selected for processing (1)
  • README.md

📝 Walkthrough

Walkthrough

The PR simplifies the README by removing Accomplish website and blog links from the top external-links section and footer area, along with the "Built by Accomplish" attribution, leaving only the releases link, issues, and Twitter links with an MIT License footer.

Changes

Documentation Updates

Layer / File(s) Summary
External links and footer cleanup
README.md
Removed Accomplish website and blog links from the top external-links section and footer; removed "Built by Accomplish" footer attribution text. The README now focuses on releases, issues, and Twitter links with an MIT License footer.

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Links removed with care so neat,
Website and blog links now retreat,
Footer trimmed to just the core,
Releases, issues, Twitter, and more—
The README breathes, much lighter and fleet! 📝✨

🚥 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 clearly and specifically describes the main change: removing brand links to www.accomplish.ai from README.md, which aligns with the primary objective of the changeset.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cleanup/remove-accomplish-ai-brand-links

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

@DanielScharfsteinGit DanielScharfsteinGit deleted the cleanup/remove-accomplish-ai-brand-links branch May 12, 2026 08:01
@DanielScharfsteinGit DanielScharfsteinGit restored the cleanup/remove-accomplish-ai-brand-links branch May 12, 2026 08:07
@DanielScharfsteinGit DanielScharfsteinGit merged commit b3140a3 into main May 12, 2026
3 of 5 checks passed
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