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

[Filesystem] Add documentation for the readlink method #6956

Merged
merged 1 commit into from
Sep 21, 2016

Conversation

tgalopin
Copy link
Contributor

@tgalopin tgalopin commented Sep 7, 2016

Following the merge of symfony/symfony#17498, this PR introduces documentation for the new methd readlink.

@javiereguiluz
Copy link
Member

@tgalopin thanks for providing this feature and for documenting it.

There are some minor syntax issues in your docs:

  • code must be wrapped with two backticks (in Markdown is just one)
  • to enable the magic PHP highlighting, the line before the PHP code block must end with two colons ::

@tgalopin tgalopin force-pushed the filesystem-readlink branch 2 times, most recently from f948414 to 37a7226 Compare September 7, 2016 12:30
@tgalopin tgalopin force-pushed the filesystem-readlink branch from 37a7226 to 3572a3d Compare September 7, 2016 12:30
@tgalopin
Copy link
Contributor Author

tgalopin commented Sep 7, 2016

Updated

@javiereguiluz
Copy link
Member

👍

Status: reviewed

@xabbuh
Copy link
Member

xabbuh commented Sep 21, 2016

Thank you @tgalopin.

@xabbuh xabbuh merged commit 3572a3d into symfony:master Sep 21, 2016
xabbuh added a commit that referenced this pull request Sep 21, 2016
…(tgalopin)

This PR was merged into the master branch.

Discussion
----------

[Filesystem] Add documentation for the readlink method

Following the merge of symfony/symfony#17498, this PR introduces documentation for the new methd `readlink`.

Commits
-------

3572a3d [Filesystem] Add documentation for the readlink method
@tgalopin tgalopin deleted the filesystem-readlink branch September 21, 2016 13:15
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