Skip to content

Release 4.0.474: reason en eventos - #7

Merged
mendozagit merged 3 commits into
mainfrom
feat/stripe-app
Nov 8, 2026
Merged

mendozagit merged 3 commits into
mainfrom
feat/stripe-app

Conversation

@mendozagit

@mendozagit mendozagit commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

Parte del release 4.0.474 (feature Stripe App). Notas completas, BREAKING primero: https://github.com/FiscalAPI/fapi-back/pull/89

Cambios

  • Eventos: reason en los sobres de fixtures y ejemplos de la documentación
  • Versión 4.0.474 (unificada con el back)

Versión y publicación

4.0.474. Release con el tag v4.0.474 (queda en borrador), después del deploy del back.

Summary by CodeRabbit

  • Documentation
    • Updated installation instructions and command-line examples to show version 4.0.474, with 4.0.475 shown as the next available update.
    • Updated event examples to include the reason field, including null where applicable, alongside existing event details.
    • Clarified that triggered events include a type-specific reason value.
    • Updated verbose output examples to display the 4.0.474 User-Agent.

@coderabbitai

coderabbitai Bot commented Nov 8, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The project version and release examples change to 4.0.474. Event documentation and test fixtures add a reason field with a null value. The trigger documentation also describes type-specific reason values.

Changes: Release version

Layer / File(s) Summary
Project version and release examples
src/FiscalApi.Cli/FiscalApi.Cli.csproj, docs/instalacion.md, docs/salida-y-codigos.md
The project version and displayed release examples change to 4.0.474. The installation update notice shows 4.0.475 available when 4.0.474 is installed.

Event reason examples

Layer / File(s) Summary
Event payload examples and fixtures
docs/listen.md, docs/trigger.md, docs/webhooks-y-eventos.md, tests/FiscalApi.Cli.Tests/Commands/EventsCommandTests.cs, tests/FiscalApi.Cli.Tests/Listen/FakeListenerApi.cs
Event examples and test fixtures add reason: null. The trigger documentation describes null for invoice.canceled and "rejectedByRecipient" for invoice.cancellation_rejected.

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Possibly related PRs

Merge Risk: Minimal · up to 7ac26

This release updates the version number, documentation examples, and test fixtures to reflect the new event reason field. No merge-blocking risk was found.

Pre-merge checks: Passed 5

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title "Release 4.0.474: reason en eventos" matches the main change in the changeset. The PR adds the reason field to event payloads in fixtures and docs, and bumps the CLI project version to 4…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
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.
Full details: Title check

Explanation

The title "Release 4.0.474: reason en eventos" matches the main change in the changeset. The PR adds the reason field to event payloads in fixtures and docs, and bumps the CLI project version to 4.0.474. The title names both the release and the reason field, so a reviewer scanning history can identify the primary change. The title is short, single-sentence, and free of noise.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 2 files. (1 skipped: 1 unsupported.)

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 318d7daf-4fed-4444-85c6-a4b3b53971b3
Commits

Reviewing files that changed from the base of the PR and between b3fb23e and 7ac264d.

Files selected for processing: 1

  • docs/trigger.md
Files skipped from review as they are similar to previous changes: 1

  • docs/trigger.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

@mendozagit
mendozagit merged commit fdcb32c into main Nov 8, 2026
2 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