Skip to content

Remove the unused Behat dependency - #14

Open
loevgaard wants to merge 1 commit into
fix/require-sylius-1.12from
chore/remove-unused-behat-dependency
Open

loevgaard wants to merge 1 commit into
fix/require-sylius-1.12from
chore/remove-unused-behat-dependency

Conversation

@loevgaard

Copy link
Copy Markdown
Member

behat/behat sits in require-dev, but there is no behat.yml, no feature files, no Behat contexts and no CI job running it. (tests/Context/ is the VatContextInterface unit tests, not Behat contexts.) The only trace of Behat in the repo is the one composer entry, which costs every contributor an install of Behat and its dependency tree for nothing.

This is the most opinionated PR of the batch — if Behat suites are on the roadmap, just close it and the dependency stays.

Stacked on #13 (both touch composer.json). Review/merge that one first; this PR retargets to master automatically once it lands.

Addresses the last nitpick in #5.

https://claude.ai/code/session_01P9NzuVPQGvaVR97HZFq98a

There is no behat.yml, no feature files, no Behat contexts and no CI job
running Behat. The only trace of it is the require-dev entry, which costs every
contributor an install of Behat and its dependency tree for nothing.

Claude-Session: https://claude.ai/code/session_01P9NzuVPQGvaVR97HZFq98a
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