Skip to content

docs: remove duplicate coach names table from README#479

Merged
nanotaboada merged 3 commits intomasterfrom
perf/remove-codacy-integration
Feb 6, 2026
Merged

docs: remove duplicate coach names table from README#479
nanotaboada merged 3 commits intomasterfrom
perf/remove-codacy-integration

Conversation

@nanotaboada
Copy link
Owner

@nanotaboada nanotaboada commented Feb 6, 2026

Replace detailed coach table with a reference to CHANGELOG.md to avoid duplication.


This change is Reviewable

Summary by CodeRabbit

  • Documentation

    • Updated README to reference CHANGELOG.md for comprehensive coach information and release history.
  • Chores

    • Consolidated coverage monitoring tools for streamlined CI/CD pipeline.
    • Simplified code quality analysis configuration.
    • Adjusted release notes formatting for improved clarity.

- Remove Codacy configuration file (.codacy.yml)
- Remove Codacy step from CI workflow (keep Codecov only)
- Remove Codacy badge from README

CodeQL handles security scanning, SonarCloud provides quality analysis,
and Codecov tracks coverage - eliminating the need for duplicate tooling.
Replace detailed coach table with a reference to CHANGELOG.md to avoid
duplication. The complete A-Z list is maintained in a single source
of truth in the changelog.
@coderabbitai
Copy link

coderabbitai bot commented Feb 6, 2026

Walkthrough

This pull request removes static analysis configuration entries and simplifies CI/CD workflows. Changes include deleting Codacy exclude paths configuration, consolidating coverage uploads to Codecov only, adjusting release notes formatting, and removing a coach names reference table from documentation.

Changes

Cohort / File(s) Summary
Static Analysis & Coverage Configuration
.codacy.yml, .github/workflows/python-ci.yml
Removed Codacy exclude_paths block entirely and simplified coverage upload workflow by replacing per-service matrix-based uploads with a single Codecov upload step, also removing Codacy coverage reporter steps.
CI/CD Release Configuration
.github/workflows/python-cd.yml
Adjusted GitHub Release body formatting by relocating the ♟️ emoji from the first line to the second line of the release header.
Documentation
README.md
Removed the A-Z Coach Names table and replaced it with a reference pointer to CHANGELOG.md for the complete coach list and release history.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • feature/coveragerc-omit #329: Modifies coverage/analysis configuration by updating omit/ignore patterns for excluded files and directories, paralleling the Codacy configuration removal in this PR.
  • feature/coverage-settings #328: Directly modifies the same .codacy.yml exclude_paths configuration that is being deleted in this PR.
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'docs: remove duplicate coach names table from README' follows Conventional Commits format, is 52 characters (well under 80), and clearly describes the main change documented in the PR objectives and raw summary.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch perf/remove-codacy-integration

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 and usage tips.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 6, 2026

@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.65%. Comparing base (0e065ee) to head (1b6c6cc).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #479   +/-   ##
=======================================
  Coverage   89.65%   89.65%           
=======================================
  Files           3        3           
  Lines         116      116           
=======================================
  Hits          104      104           
  Misses         12       12           
Components Coverage Δ
Services 79.31% <ø> (ø)
Routes 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@nanotaboada nanotaboada merged commit 3e85aa0 into master Feb 6, 2026
15 checks passed
@nanotaboada nanotaboada deleted the perf/remove-codacy-integration branch February 6, 2026 05:32
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