Skip to content

docs(changelog): prepare release notes for v1.1.2-drogba#107

Merged
nanotaboada merged 2 commits into
masterfrom
release/v1.1.2-drogba
May 5, 2026
Merged

docs(changelog): prepare release notes for v1.1.2-drogba#107
nanotaboada merged 2 commits into
masterfrom
release/v1.1.2-drogba

Conversation

@nanotaboada

@nanotaboada nanotaboada commented May 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • Aligns the /pre-release command's minor/patch bump rule with go-samples-gin-restful: ### Added entries only trigger a minor bump when they introduce a new endpoint, request parameter, or client-visible API response; test additions and internal tooling qualify as patch
  • Prepares release notes for v1.1.2-drogba (patch): field-level validation via the validator crate and six new integration tests covering 422 responses for POST and PUT

What's included in this release

  • Field-level validation on PlayerRequest payloads (validator crate, #97): required string fields must be non-empty; squad_number must be in range 1–99
  • Six new route-level integration tests for validation failure scenarios on POST and PUT (#97)
  • POST /players and PUT /players/squadnumber/{squad_number} now return 422 Unprocessable Entity for field validation failures; 400 Bad Request reserved for malformed JSON / wrong Content-Type (#94, #97)

Test plan

  • CI passes (fmt, clippy, tests, build)
  • CHANGELOG diff is correct: [Unreleased] is empty, [1.1.2 - Drogba] section is present with today's date
  • Compare links at the bottom of CHANGELOG are correct
  • No regressions in existing tests

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation

    • Released version 1.1.2 with field-level validation rules and refined HTTP status codes: validation errors return 422, malformed JSON returns 400.
  • Chores

    • Updated release automation to improve breaking change detection and refine version bump criteria.

@coderabbitai

coderabbitai Bot commented May 5, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d9b416ee-32bb-480e-9f67-ca14a3506cf4

📥 Commits

Reviewing files that changed from the base of the PR and between 2d374e6 and d8f6c69.

📒 Files selected for processing (2)
  • .claude/commands/pre-release.md
  • CHANGELOG.md

Walkthrough

Updated version-bump detection rules to explicitly identify breaking changes (BREAKING, BREAKING CHANGE:, or ! suffix) as major bumps and narrow minor bumps to true API additions. Populated the Unreleased section structure and created the 1.1.2 release entry documenting PlayerRequest validation and updated HTTP status behavior (422 for validation failures).

Changes

Release Documentation & Rules

Layer / File(s) Summary
Version Bump Rules
.claude/commands/pre-release.md
Major bump detection refined for breaking-change signals (BREAKING, BREAKING CHANGE:, or ! suffix); minor bump narrowed to true client-visible API additions (new endpoint/parameter/response); patch bump as default for non-breaking additions.
Changelog Entry
CHANGELOG.md
Unreleased section structured with empty subsections for Changed, Fixed, Removed; [1.1.2 - Drogba] release documents PlayerRequest field-level validation rules, integration tests, and HTTP 422 status for field validation failures (with 400 reserved for malformed JSON/wrong Content-Type); unreleased compare URL updated from v1.1.1-cannavaro to v1.1.2-drogba.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed Title follows Conventional Commits format with 'docs' prefix, is under 80 characters (56 chars), and clearly describes the main change: preparing release notes for v1.1.2-drogba.
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
  • Commit unit tests in branch release/v1.1.2-drogba
  • 🛠️ sync documentation: Commit on current branch
  • 🛠️ sync documentation: Create PR
  • 🛠️ enforce http error handling: Commit on current branch
  • 🛠️ enforce http error handling: Create PR
  • 🛠️ idiomatic review: Commit on current branch
  • 🛠️ idiomatic review: Create PR
  • 🛠️ verify api contract: Commit on current branch
  • 🛠️ verify api contract: Create PR

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.

@codecov

codecov Bot commented May 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.88%. Comparing base (2d374e6) to head (d8f6c69).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #107   +/-   ##
=======================================
  Coverage   84.88%   84.88%           
=======================================
  Files           6        6           
  Lines         172      172           
=======================================
  Hits          146      146           
  Misses         26       26           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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 and others added 2 commits May 5, 2026 11:39
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
@nanotaboada
nanotaboada force-pushed the release/v1.1.2-drogba branch from 46c33d2 to d8f6c69 Compare May 5, 2026 14:40
@nanotaboada
nanotaboada merged commit 33d7aff into master May 5, 2026
10 checks passed
@nanotaboada
nanotaboada deleted the release/v1.1.2-drogba branch May 5, 2026 14:55
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