Skip to content

Command fix in upgrade_major.rst #13547

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

Closed
wants to merge 0 commits into from
Closed

Conversation

antiseptikk
Copy link
Contributor

Hi, without the double quotes i've got no matches found: symfony/* error.

@antiseptikk antiseptikk changed the title Update upgrade_major.rst Command fix in upgrade_major.rst Apr 22, 2020
@javiereguiluz javiereguiluz added this to the 4.4 milestone Jun 1, 2020
@javiereguiluz javiereguiluz changed the base branch from master to 4.4 June 1, 2020 06:48
@javiereguiluz
Copy link
Member

@antiseptikk thanks a lot for this contribution. Sadly, when I tried to merge it with our proprietary tool, it failed (and the tool closed the pull request automatically). I also tried to merge #13556 but it failed in the same way. It turns out that the article had already been updated (you can see "symfony/*" wrapped in quotes in https://symfony.com/doc/current/setup/upgrade_major.html).

I'm sorry and thanks again.

@xabbuh
Copy link
Member

xabbuh commented Jun 1, 2020

FYI, the fix was merged with #13610.

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.

4 participants