Skip to content

chore: remove Glama integration#103

Merged
cmeans-claude-dev[bot] merged 1 commit into
mainfrom
chore/remove-glama
May 2, 2026
Merged

chore: remove Glama integration#103
cmeans-claude-dev[bot] merged 1 commit into
mainfrom
chore/remove-glama

Conversation

@cmeans-claude-dev
Copy link
Copy Markdown
Contributor

@cmeans-claude-dev cmeans-claude-dev Bot commented May 2, 2026

Summary

Drops the README Glama score badge and the glama.json registry metadata file.

Removed

  • README Glama score badge (line 17)
  • glama.json
  • README "Project layout" line referencing glama.json

CHANGELOG

The in-Unreleased badge swap from PR #100 (card.svg -> score.svg) is moot now that the badge itself is gone, so the ### Changed entry is replaced with a new ### Removed section under the Keep-a-Changelog ordering.

Test plan

  • uv run pytest (504 passed, 6 deselected, 5 xfailed; no test changes here)
  • uv run ruff check src/ tests/
  • uv run ruff format --check src/ tests/
  • Visual check the README renders without a broken-image gap where the badge used to be

CHANGELOG

Drop the README Glama score badge and the glama.json registry metadata
file. Supersedes the in-Unreleased badge swap from PR #100 (which is
now moot since the badge itself is gone). README structure listing in
"Project layout" updated to drop the glama.json line.

CHANGELOG: replace the obsolete `### Changed` swap entry with a new
`### Removed` section under Keep-a-Changelog ordering.
@github-actions github-actions Bot added the Awaiting CI Dev complete, waiting for CI/Codecov to pass before QA label May 2, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown
Owner

@cmeans cmeans left a comment

Choose a reason for hiding this comment

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

LGTM

@cmeans cmeans added QA Active QA is actively reviewing; Dev should not push changes and removed Awaiting CI Dev complete, waiting for CI/Codecov to pass before QA labels May 2, 2026
@cmeans
Copy link
Copy Markdown
Owner

cmeans commented May 2, 2026

[QA] Starting review at c72debd. Note: CI was green when I picked this up but the Awaiting CI label was still set — applying QA Active to clear the stale state. Will exercise the README Glama removal, glama.json deletion, and CHANGELOG ordering, plus a repo-wide grep for residual references.

Copy link
Copy Markdown
Owner

@cmeans cmeans left a comment

Choose a reason for hiding this comment

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

QA review — round 1

Clean docs/cleanup PR at c72debd. Diff is +5/-11 across CHANGELOG.md, README.md, and glama.json (deleted). Zero findings.

Verification

Check Status
uv run pytest 504 passed, 6 deselected, 5 xfailed
uv run ruff check src/ tests/ clean
uv run ruff format --check src/ tests/ 9 files already formatted
uv run mypy src/ clean (not in PR test plan since this is docs-only, ran anyway as a safety net)
Repo-wide grep glama 1 match outside the diff: .claude/settings.local.json:45 (WebFetch(domain:glama.ai) permission). Confirmed untracked (git ls-files empty for .claude/); dev-local Claude Code settings, not part of the project. Out of scope.
README badge layout core badges (PyPI / Python versions / License / CI / Coverage / Downloads) on lines 11–16; previously line 17 was Glama; now line 17 is blank, line 18 is the pip-downloads badge group. No broken-image gap, the natural blank-line separator between badge groups already covered the spacing.
README "Project layout" glama.json line at the old line 308 is gone; surrounding tree (CHANGELOG.md / CLAUDE.md / LICENSE / README.md) renders correctly.
CHANGELOG ordering KaC v1.1.0 order preserved: Added → Changed → Removed (new) → Fixed → Security. The obsolete card.svg → score.svg badge-swap line under ### Changed is correctly retired (it described a swap on a now-deleted badge).
CHANGELOG entry traceability new ### Removed bullet ends with (#103)
CI gates all green (lint, typecheck, test 3.11/3.12/3.13, codecov/patch, qa-approved)

Workflow note

Picked this up while the label was still Awaiting CI even though every CI check was already SUCCESS — the pr-labels-ci.yml workflow_run promotion to Ready for QA hadn't fired yet (likely a small dispatcher delay; QA Gate StatusContext was still PENDING). Applying QA Active cleared the stale label as a side effect of the automation. Worth keeping an eye on if it recurs across subsequent PRs.

Applying Ready for QA Signoff as the final act. Awaiting maintainer QA Approved.

@cmeans
Copy link
Copy Markdown
Owner

cmeans commented May 2, 2026

[QA] Verdict: Ready for QA Signoff. Single round, zero findings. Gates re-verified at c72debd (504/6/5, ruff/format/mypy clean, CI green); README layout and CHANGELOG ordering intact; only out-of-scope match for "glama" is in untracked dev-local .claude/settings.local.json. Awaiting maintainer QA Approved.

@cmeans cmeans added Ready for QA Signoff QA passed — ready for maintainer final review and merge QA Approved Manual QA testing completed and passed and removed QA Active QA is actively reviewing; Dev should not push changes Ready for QA Signoff QA passed — ready for maintainer final review and merge labels May 2, 2026
@cmeans-claude-dev cmeans-claude-dev Bot merged commit 7ac9a20 into main May 2, 2026
36 checks passed
@cmeans-claude-dev cmeans-claude-dev Bot deleted the chore/remove-glama branch May 2, 2026 01:35
@cmeans-claude-dev cmeans-claude-dev Bot mentioned this pull request May 2, 2026
6 tasks
cmeans-claude-dev Bot added a commit that referenced this pull request May 3, 2026
Bump pyproject.toml 2.2.1 -> 2.3.0 and convert the [Unreleased] block into [2.3.0] - 2026-05-02. A fresh empty [Unreleased] section sits above for the next cycle.

13 PRs aggregated since v2.2.1: #88, #92, #93, #94, #95, #96, #98, #99, #100, #101, #102, #103, #104.

Tag-push (v2.3.0) after merge triggers .github/workflows/publish.yml.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

QA Approved Manual QA testing completed and passed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant