Skip to content

fix name of the Yaml component #4199

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

Merged
merged 1 commit into from
Sep 15, 2014
Merged

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Sep 5, 2014

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets related to #4189

The component name that is used in the code as well as in the issue tracker used by the symfony/symfony repository is Yaml (instead of YAML). Therefore, this should also be the name being used in the documentation.

@javiereguiluz
Copy link
Member

👍 thanks @xabbuh for bringing in consistency!

However, there is a YAML instance that we could also transform to Yaml. It's in this phrase:

Fast

One of the goal of Symfony YAML is to find the right balance between speed and features. It supports just the needed feature to handle configuration files.

Symfony YAML refers to the component or to the subset dialect of YAML supported by Symfony?

The component name that is used in the code as well as in the issue
tracker used by the symfony/symfony repository is Yaml (instead of
YAML). Therefore, this should also be the name being used in the
documentation.
@xabbuh xabbuh force-pushed the yaml-component-name branch from 26c3ac5 to 5187768 Compare September 5, 2014 09:50
@xabbuh
Copy link
Member Author

xabbuh commented Sep 5, 2014

@javiereguiluz Good catch. The sentence is also worded really weird. I updated it too.

@javiereguiluz
Copy link
Member

@xabbuh it's much better now! Thanks.

@weaverryan
Copy link
Member

Cool, thanks @xabbuh!

@weaverryan weaverryan merged commit 5187768 into symfony:2.3 Sep 15, 2014
weaverryan added a commit that referenced this pull request Sep 15, 2014
This PR was merged into the 2.3 branch.

Discussion
----------

fix name of the Yaml component

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets | related to #4189

The component name that is used in the code as well as in the issue tracker used by the symfony/symfony repository is Yaml (instead of YAML). Therefore, this should also be the name being used in the documentation.

Commits
-------

5187768 fix name of the Yaml component
@xabbuh xabbuh deleted the yaml-component-name branch September 15, 2014 14:47
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