Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update framework.rst #9043

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 2 commits into from
Closed

Conversation

MatthieuBraure
Copy link
Contributor

Add atom editor to IDE config.

Add atom editor to IDE config.
@@ -204,7 +204,7 @@ ide
Symfony turns file paths seen in variable dumps and exception messages into
links that open those files right inside your browser. If you prefer to open
those files in your favorite IDE or text editor, set this option to any of the
following values: ``phpstorm``, ``sublime``, ``textmate``, ``macvim`` and ``emacs``.
following values: ``phpstorm``, ``sublime``, ``textmate``, ``macvim``, ``emacs`` and ``atom``.
Copy link
Member

Choose a reason for hiding this comment

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

Nice! Thanks for your contribution. I just have a very minor comment. To make docs easier to read out of the symfony.com website, we limit the length of the lines of text to approximately 80 characters. So, could you please edit this change and move and atom. to a new line? Thanks!


Note: since I don't know if you have much or little experience with GitHub, just in case you need it, you can change the pull request using GitHub interface by clicking on this icon:

github-edit-changes

Move 'and atom' to a new line.
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.

Great! This is now ready for merge. I'm really happy to have a new Symfony Docs contributor. Thank you!

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.

3 participants