Skip to content

Conversation

snoek09
Copy link

@snoek09 snoek09 commented Jan 29, 2017

Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

👍

Nice catch @snoek09. Thanks!

@xabbuh
Copy link
Member

xabbuh commented Jan 31, 2017

Good catch, thanks @snoek09.

@xabbuh xabbuh merged commit 27a7172 into symfony:3.1 Jan 31, 2017
xabbuh added a commit that referenced this pull request Jan 31, 2017
This PR was merged into the 3.1 branch.

Discussion
----------

remove unused parameter from code example

Small fix for code example in https://symfony.com/doc/current/frontend/custom_version_strategy.html#implement-versionstrategyinterface.

Commits
-------

27a7172 remove unused options parameter from loadManifest
@snoek09 snoek09 deleted the remove-options-parameter-from-code-example branch January 31, 2017 21:30
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