Skip to content

Minor improvement in the node types explanation #5044

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 4 commits into from
Mar 14, 2015

Conversation

javiereguiluz
Copy link
Member

Q A
Doc fix? no
New docs? yes
Applies to 2.3+
Fixed tickets -

@timglabisch
Copy link
Contributor

👍

* boolean
* integer (new in 2.2)
* float (new in 2.2)
* enum (new in 2.1)
* enum (new in 2.1) (similar to array, but it only allows a finite set of values)
Copy link
Member

Choose a reason for hiding this comment

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

it is similar to scalar, not to array

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed. Thanks.

@javiereguiluz
Copy link
Member Author

The errors reported by reviewers have been addressed and there are no new comments. I think we can safely tag this PR as finished.

@weaverryan
Copy link
Member

Thanks Javier!

@weaverryan weaverryan merged commit 987f5a5 into symfony:2.3 Mar 14, 2015
weaverryan added a commit that referenced this pull request Mar 14, 2015
…uiluz)

This PR was merged into the 2.3 branch.

Discussion
----------

Minor improvement in the node types explanation

| Q             | A
| ------------- | ---
| Doc fix?      | no
| New docs?     | yes
| Applies to    | 2.3+
| Fixed tickets | -

Commits
-------

987f5a5 "enum" is similar to "scalar", not "array"
96c423a Moved the admonition about the INI loader to the loaders section
c774b8a Improved a bit the information about dumping config
daff271 Minor improvement in the node types explanation
@javiereguiluz javiereguiluz deleted the improve_config branch January 3, 2018 16:34
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.

6 participants