Skip to content

Conversation

@OskarStark
Copy link
Contributor

Q A
Bug fix? no
New feature? no
Docs? no
Issues --
License MIT

The store bridges have their own composer.json with their specific dependencies. The main store package doesn't need these vendor packages in require-dev since it only tests core functionality.

@carsonbot carsonbot added Store Issues & PRs about the AI Store component Status: Needs Review labels Dec 15, 2025
@OskarStark OskarStark force-pushed the OskarStark/store-deps-review branch 7 times, most recently from be004e4 to 7210097 Compare December 15, 2025 17:31
@OskarStark OskarStark force-pushed the OskarStark/store-deps-review branch from 7210097 to f9f07ff Compare December 15, 2025 17:34
@OskarStark OskarStark merged commit 69ed9b6 into symfony:main Dec 15, 2025
OskarStark added a commit that referenced this pull request Dec 15, 2025
…igs (OskarStark)

This PR was merged into the main branch.

Discussion
----------

[Store] Include `Tests/` directory in Bridge PHPStan configs

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| Docs?         | no
| Issues        | Forgotten in #1133
| License       | MIT

Commits
-------

d0f2f9d [Store] Include Tests/ directory in Bridge PHPStan configs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Needs Review Store Issues & PRs about the AI Store component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants