Skip to content

Conversation

@adrienbrault
Copy link

@adrienbrault adrienbrault commented Jul 2, 2022

This is #996 rebased. I've removed the change to CHANGELOG.md.

I've been using this patch for years now, it's great!

@adrienbrault adrienbrault marked this pull request as ready for review July 2, 2022 13:35
@Chekote
Copy link
Contributor

Chekote commented Jul 6, 2022

@adrienbrault you'll want to rebase this so you get the fix for the issue that failed your build.

@adrienbrault
Copy link
Author

@Chekote This PR targets 4.x ( #996 (comment) ), which does not have the fix, yet.

@ciaranmcnulty
Copy link
Contributor

I'll merge that up and rebase this branch when I get time 👍

@ciaranmcnulty ciaranmcnulty force-pushed the feature/multiple-paths branch from 7503223 to 9995fdb Compare July 7, 2022 11:58
@ciaranmcnulty ciaranmcnulty added this to the 4.0.0 milestone Jul 7, 2022
@ciaranmcnulty ciaranmcnulty changed the title Accept multiple path arguments Accept multiple path arguments at CLI Jul 7, 2022
@ciaranmcnulty
Copy link
Contributor

Thanks for your patience @adrienbrault

@ciaranmcnulty ciaranmcnulty merged commit 9b86a08 into Behat:4.x Jul 7, 2022
acoulton added a commit to acoulton/Behat that referenced this pull request Nov 11, 2025
Brings the history of the old abandoned 4.x branch into the current
master, ahead of creating a new 4.x branch. This will avoid us losing
the record of these contributions when we restart development for 4.0.

There are only very limited changes - there was only one functional PR
Behat#1397 that was merged into 4.x in 2023. The same feature was
subsequently implemented by a different contributor in Behat#1611, merged
into master and released in 3.23.0.

The solutions are essentially identical except for a change to
WrongPathsException which I have kept both because it is more correct
and in case anyone is already using the 4.x branch.
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.

3 participants