Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
Yaml: Removed boolean flags#6950
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
components/yaml.rst Outdated
| Yaml::parse($yaml,true); | ||
| Yaml::parse($yaml,Yaml::PARSE_EXCEPTION_ON_INVALID_TYPE); | ||
| However, you can activate object support using the next argument:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
this is no longer true, it's not the next statement. Can you please rewrite this sentence?
wouterj commentedSep 3, 2016
Thanks for catching this and fixing it! I've left one minor comment. If you can fix it, it would be great. Otherwise, we'll do it while merging. Status: needs work |
xabbuh commentedSep 3, 2016 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
For the records: This will fix parts (everything?) of#6226. |
chr-hertel commentedSep 3, 2016
wouterj commentedSep 3, 2016
No worries, you're doing a perfect job! :) |
components/yaml.rst Outdated
| foo:bar | ||
| bar:baz | ||
| Mulit-line strings as scalar blocks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Mulit-line ->Multi-line
javiereguiluz commentedSep 5, 2016
@chr-hertel I left some minor comments but your pull request is great! Thank you. 👍 Status: reviewed |
chr-hertel commentedSep 5, 2016
thanks@javiereguiluz, i fixed that. |
xabbuh commentedSep 21, 2016 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@chr-hertel Sorry for the late feedback: It looks like we covered parts of this with#6622. Can you rebase here and resolve the conflicts? Status: Needs work |
HeahDude commentedApr 1, 2017
Hello@chr-hertel do you still have time to work on this or do you want us to finish it? Thanks! |
ghost commentedApr 8, 2017
i guess this one is not longer needed. |
Remove deprecated boolean flags and use const instead.