Skip to content

[Finder] Fix gitignore regex build with "**" #41241

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

Merged
merged 1 commit into from
May 16, 2021

Conversation

mvorisek
Copy link
Contributor

Q A
Branch? 4.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets Fix #41223
License MIT
Doc PR no

covered with tests, also faster, we do not have to explode the rule at all :)

ping @lindelius @OskarStark

@derrabus
Copy link
Member

Thank your for your PR. Two questions:

  • Did you target the 5.x branch on purpose? The PR discription says you wanted to target 4.4.
  • Can we find a PR title that people would want to read in a ChangeLog? 🙂

@carsonbot carsonbot changed the title Fix 41223 issue "**/vendor/**" [Finder] Fix 41223 issue "**/vendor/**" May 15, 2021
@mvorisek mvorisek force-pushed the fix_gitignore_41223 branch from b88efcb to 27e2c59 Compare May 15, 2021 18:43
@mvorisek mvorisek changed the base branch from 5.x to 4.4 May 15, 2021 18:44
@mvorisek mvorisek force-pushed the fix_gitignore_41223 branch from 27e2c59 to 36ee1fe Compare May 15, 2021 19:00
@mvorisek mvorisek changed the title [Finder] Fix 41223 issue "**/vendor/**" [Finder] Fix gitignore with "**" May 15, 2021
@mvorisek mvorisek changed the title [Finder] Fix gitignore with "**" [Finder] Fix gitignore regex build with "**" May 15, 2021
@mvorisek
Copy link
Contributor Author

@derrabus base changed, it should be 4.4, title also updated

@mvorisek mvorisek force-pushed the fix_gitignore_41223 branch 4 times, most recently from 396c1b4 to d91eafb Compare May 15, 2021 20:00
@lindelius
Copy link

I can confirm that this PR fixes the issue for us, at least. Thanks!

@fabpot fabpot force-pushed the fix_gitignore_41223 branch from d91eafb to ce22d5d Compare May 16, 2021 12:25
@fabpot
Copy link
Member

fabpot commented May 16, 2021

Thank you @mvorisek.

@fabpot fabpot merged commit 03519d4 into symfony:4.4 May 16, 2021
@mvorisek mvorisek deleted the fix_gitignore_41223 branch May 16, 2021 12:37
This was referenced May 19, 2021
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.

Symfony/finder no longer handles patterns like **/vendor/**
6 participants