Skip to content

[cnpj-gen] Upgrade dependencies and deprecate resources#44

Merged
juliolmuller merged 4 commits into
mainfrom
feat/cnpj-gen-v2.1
May 26, 2026
Merged

[cnpj-gen] Upgrade dependencies and deprecate resources#44
juliolmuller merged 4 commits into
mainfrom
feat/cnpj-gen-v2.1

Conversation

@juliolmuller
Copy link
Copy Markdown
Collaborator

@juliolmuller juliolmuller commented May 26, 2026

Summary by CodeRabbit

  • Improvements

    • Runtime dependency bumped to v1.1, yielding a slight performance improvement.
  • Deprecations

    • CnpjType is deprecated and will be renamed to CnpjGenerationType in the next major release. Adopt an import alias now to ensure forward compatibility.

Review Change Stack

@juliolmuller juliolmuller requested a review from a team as a code owner May 26, 2026 13:28
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: bf35355c-5d11-4fe2-ab6a-bfd0b2871624

📥 Commits

Reviewing files that changed from the base of the PR and between 272e77e and 1e45830.

📒 Files selected for processing (1)
  • packages/cnpj-gen/CHANGELOG.md

📝 Walkthrough

Walkthrough

This PR updates the CNPJ generator: bumps the runtime dependency lacus/cnpj-dv to v1.1, adds a v2.1.0 CHANGELOG entry, and marks the CnpjType enum deprecated pending a rename to CnpjGenerationType.

Changes

v2.1.0 Release Preparation

Layer / File(s) Summary
Release documentation and dependency update
packages/cnpj-gen/CHANGELOG.md, packages/cnpj-gen/composer.json
CHANGELOG adds ## 2.1.0 with Improvements and Deprecations; composer.json updates lacus/cnpj-dv from ^1.0 to ^1.1.
Enum deprecation annotation
packages/cnpj-gen/src/Enums/CnpjType.php
Adds PHPDoc deprecation to CnpjType indicating it will be renamed to CnpjGenerationType in the next major version and recommends using an import alias for compatibility.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

enhancement

Poem

🐰 A dependency blooms, a version takes flight,
CNPJ hums swifter through day and night.
CnpjType whispers change, a gentle guide,
Alias your imports, let futures coincide.

🚥 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 accurately summarizes the main changes: upgrading the lacus/cnpj-dv dependency and adding a deprecation notice to CnpjType.
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 feat/cnpj-gen-v2.1

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.

@juliolmuller juliolmuller added enhancement New minor or major features. dependencies Changes to the project dependencies. labels May 26, 2026
@coderabbitai coderabbitai Bot changed the title @coderabbitai Upgrade lacus/cnpj-dv to v1.1 and deprecate CnpjType enum May 26, 2026
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@packages/cnpj-gen/CHANGELOG.md`:
- Line 11: The changelog line about CnpjType is ambiguous; update the sentence
to explicitly state when it was deprecated and the planned rename by replacing
“was deprecated in and will be renamed” with wording such as “deprecated since
v2.1.0 and will be renamed in the next major version,” and keep the suggested
import alias example referencing CnpjType and CnpjGenerationType to show the
migration path.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 2b813d94-a1df-43d8-af20-8e8a7da20a4a

📥 Commits

Reviewing files that changed from the base of the PR and between 06a739a and 272e77e.

📒 Files selected for processing (3)
  • packages/cnpj-gen/CHANGELOG.md
  • packages/cnpj-gen/composer.json
  • packages/cnpj-gen/src/Enums/CnpjType.php

Comment thread packages/cnpj-gen/CHANGELOG.md Outdated
@juliolmuller juliolmuller changed the title Upgrade lacus/cnpj-dv to v1.1 and deprecate CnpjType enum [cnpj-gen] Upgrade dependencies and deprecate resources May 26, 2026
Fix as per CodeRabbit AI review comment at #44 (comment).

Co-authored-by: CodeRabbit AI <support@coderabbit.ai>
@juliolmuller juliolmuller merged commit 09b3b6b into main May 26, 2026
100 checks passed
@juliolmuller juliolmuller deleted the feat/cnpj-gen-v2.1 branch May 26, 2026 15:31
juliolmuller added a commit that referenced this pull request May 26, 2026
Fix as per CodeRabbit AI review comment at #44 (comment).

Co-authored-by: CodeRabbit AI <support@coderabbit.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Changes to the project dependencies. enhancement New minor or major features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant