Skip to content
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

Remove support for Behat Sf Extension #832

Merged
1 commit merged into from Oct 21, 2020

Conversation

Nek-
Copy link
Contributor

@Nek- Nek- commented Oct 7, 2020

The problem here is that the Symfony extension for Behat is not maintained and not compatible with new versions of Symfony. It has been officialized here:
Behat/Symfony2Extension#152 (comment)

The new correct way to use Behat with Symfony is to use the package Symfony extension from the FriendsOfBehat organization.

Q A
License MIT
Doc issue/PR N/A

Oh and btw if you merge it you can add the hacktoberfest-accepted label :) .

The problem here is that the symfony extension for behat is not
maintained and not compatible with new versions of Symfony. It has been
officialized here:
Behat/Symfony2Extension#152 (comment)

The new correct way to use behat with Symfony is to use the package
symfony extension from the FriendsOfBehat organization.
- https://github.com/FriendsOfBehat/SymfonyExtension
- https://github.com/symfony/recipes-contrib/tree/master/friends-of-behat/symfony-extension/2.0
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request passes validation.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request passes validation.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request passes validation.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request passes validation.

@nicolas-grekas
Copy link
Member

Should we move it to recipes-contrib? Can you please do the PR if yes?

@Nek-
Copy link
Contributor Author

Nek- commented Oct 7, 2020

@nicolas-grekas no, it should just not exist. It already exists a recipe for the correct package to use. Even Google suggests the good one first.

image
Please notice that the search "Behat Symfony Extension" does not result correctly but I assume we can't do anything for it.

@ghost ghost merged commit f9e664d into symfony:master Oct 21, 2020
@wouterj
Copy link
Member

wouterj commented Oct 21, 2020

Btw, I think we should change the behat alias to the new Friends of Behat package and include those recipes in this repository.
The Behat team has moved all extension to the Friends of Behat organization.

I'm happy to provide the move PR if this is supported, not sure how to add the behat alias.

@Nek- Nek- deleted the remove-behat-extension branch October 21, 2020 21:05
This pull request was closed.
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.

None yet

4 participants