Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[2.7] update readme files for new components#18044
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
xabbuh commentedMar 7, 2016
Oh, we need to take into account the split of the Security component too. Status: Needs Work |
| echo $language->compile('"foo" ~ upper(foo)'); | ||
| // would output ("foo" . strtoupper($foo)) | ||
| evaluate expressions. An expression is a one-liner that returns a value | ||
| (mostly, but not limited to, Booleans). |
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.
Booleans and scalars ?
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.
I reused the introduction from the documentation:http://symfony.com/doc/current/components/expression_language/introduction.html
I think if we want to improve this (which doesn't seem to be the worst idea), we should make the change there first.
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.
I agree with@xabbuh. Let's copy the current description to move forward fast and let's improve it in the future.
xabbuh commentedMar 7, 2016
Security sub-components updated too. Status: Needs Review |
src/Symfony/Bridge/PhpUnit/README.md Outdated
| Resources | ||
| --------- | ||
| *[Documentation](http://symfony.com/doc/current/components/phpunit_bridge.html) |
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.
Please usehttps:// in this link.
xabbuh commentedMar 7, 2016
@javiereguiluz Thanks for the heads up. Fixed all the links. |
javiereguiluz commentedMar 7, 2016
👍 Status: Reviewed |
nicolas-grekas commentedMar 7, 2016
Thank you@xabbuh. |
This PR was merged into the 2.7 branch.Discussion----------[2.7] update readme files for new components| Q | A| ------------- | ---| Branch | 2.7| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |#17997| License | MIT| Doc PR |This completes@javiereguiluz's great work from#17997 by adapting the readme files of components added after the release of Symfony 2.3.Commits-------5ba194e [2.7] update readme files for new components
This completes@javiereguiluz's great work from#17997 by adapting the readme files of components added after the release of Symfony 2.3.