Skip to content

Add verbose mode to ReEncryptModels command#10

Merged
JaZo merged 3 commits intomasterfrom
feature/verbose-mode
Oct 20, 2025
Merged

Add verbose mode to ReEncryptModels command#10
JaZo merged 3 commits intomasterfrom
feature/verbose-mode

Conversation

@JaZo
Copy link
Member

@JaZo JaZo commented Oct 18, 2025

Description

I added more output to the ReEncryptModels command. It now outputs each model key in verbose mode and each attribute in very verbose mode.

Motivation and context

So you can see what the command is doing.

How has this been tested?

Updated tests.

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have read the CONTRIBUTING document.
  • My pull request addresses exactly one patch/feature.
  • I have created a branch for this patch/feature.
  • Each individual commit in the pull request is meaningful.
  • I have added tests to cover my changes.
  • If my change requires a change to the documentation, I have updated it accordingly.

@JaZo
Copy link
Member Author

JaZo commented Oct 18, 2025

Tests fail on symfony/console below 7.3.3. https://symfony.com/blog/symfony-7-3-3-released

@bbrala
Copy link
Member

bbrala commented Oct 18, 2025

Code looks good. Failures are unfortunate. You using aoecific features?

@JaZo
Copy link
Member Author

JaZo commented Oct 20, 2025

I think the failure is caused by this issue: symfony/symfony#61033. I'll bump the minimum version for dev so tests pass.

@JaZo JaZo merged commit 3c7f8c5 into master Oct 20, 2025
10 of 11 checks passed
@JaZo JaZo deleted the feature/verbose-mode branch October 20, 2025 07:45
@JaZo JaZo added this to the 3.0 milestone Oct 21, 2025
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.

2 participants