Skip to content

[FrameworkBundle] Add functional tests for ContainerLintCommand #60954

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: 6.4
Choose a base branch
from

Conversation

santysisi
Copy link
Contributor

@santysisi santysisi commented Jun 29, 2025

Q A
Branch? 6.4
Bug fix? no
New feature? no
Deprecations? no
Issues no
License MIT

This PR adds a new functional test class for the lint:container command. It addresses a gap from a previous PR where the command logic was updated to fix bugs but no corresponding tests were included due to time constraints.

@carsonbot carsonbot added this to the 7.4 milestone Jun 29, 2025
@santysisi santysisi force-pushed the minor/add-container-lint-command-test branch 2 times, most recently from ed4c33e to 56513a8 Compare June 30, 2025 02:01
@OskarStark OskarStark changed the title [FrameworkBundle] Add functional tests for ContainerLintCommand [FrameworkBundle] Add functional tests for ContainerLintCommand Jun 30, 2025
@xabbuh
Copy link
Member

xabbuh commented Jun 30, 2025

For 6.4?

@OskarStark OskarStark modified the milestones: 7.4, 6.4 Jun 30, 2025
@santysisi santysisi changed the base branch from 7.4 to 6.4 June 30, 2025 14:07
@santysisi santysisi changed the base branch from 6.4 to 7.4 June 30, 2025 14:07
@santysisi santysisi force-pushed the minor/add-container-lint-command-test branch from 56513a8 to d997225 Compare June 30, 2025 14:37
@santysisi santysisi changed the base branch from 7.4 to 6.4 June 30, 2025 14:37
@santysisi
Copy link
Contributor Author

Sorry for the unnecessary review request my mistake 🙏

@santysisi santysisi force-pushed the minor/add-container-lint-command-test branch from d997225 to f1971a2 Compare July 1, 2025 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants