Skip to content

Conversation

kick-the-bucket
Copy link
Contributor

@kick-the-bucket kick-the-bucket commented Jan 4, 2018

Q A
Branch? master
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

Since running composer commands is time heavy, having a wrong one recommended to you just burns your time.

Since running composer commands is time heavy, having a wrong one recommended to you just burns your time.
Copy link
Member

@nicolas-grekas nicolas-grekas left a comment

Choose a reason for hiding this comment

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

(for 3.4)

@nicolas-grekas nicolas-grekas added this to the 3.4 milestone Jan 4, 2018
@fabpot fabpot changed the base branch from master to 3.4 January 4, 2018 13:03
@fabpot fabpot changed the base branch from 3.4 to master January 4, 2018 13:04
@fabpot
Copy link
Member

fabpot commented Jan 4, 2018

Good catch, thanks @kick-the-bucket.

fabpot added a commit that referenced this pull request Jan 4, 2018
…vendor) (Tomas Kmieliauskas)

This PR was submitted for the master branch but it was merged into the 3.4 branch instead (closes #25680).

Discussion
----------

[FrameworkBundle] Fix recommended composer command (add vendor)

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        |

Since running composer commands is time heavy, having a wrong one recommended to you just burns your time.

Commits
-------

f6d38b3 [FrameworkBundle] Fix recommended composer command (add vendor)
@fabpot fabpot closed this Jan 4, 2018
@kick-the-bucket kick-the-bucket deleted the register-security-csrf-configuration branch January 8, 2018 14:18
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