-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Small type and language changes in testing client (client_typo branch) #2
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Keep it that way, that's easier for everybody; small branches/changes/pull requests. Thanks. |
weaverryan
added a commit
that referenced
this pull request
Dec 23, 2013
Improvements to registering an extension (#2)
fabpot
referenced
this pull request
in fabpot/symfony-docs
Nov 12, 2014
Commits ------- b239305 Fixed typo Discussion ---------- Fixed typo
weaverryan
added a commit
that referenced
this pull request
Dec 26, 2014
This PR was submitted for the master branch but it was merged into the 2.3 branch instead (closes #4526). Discussion ---------- Deploy Symfony application on Platform.sh. Commits ------- 105088a Update map.rst.inc 257e9a8 Coding standard for PHP block. 18ca974 Fix coding standards #2 29988aa Use alphabetical order. 75a1c56 Fix duplicated references. 8fb400e Add some references. a8b6df8 Reference available services 62f76bf Better match Symfony documentation standard. b70f413 Github => GitHub c2a5675 Fix typo and add Github link. b4383b5 Deploy Symfony application on Platform.sh.
weaverryan
added a commit
that referenced
this pull request
Dec 26, 2014
* 2.3: Update map.rst.inc Coding standard for PHP block. Fix coding standards #2 Use alphabetical order. Fix duplicated references. Add some references. Reference available services Better match Symfony documentation standard. Github => GitHub Fix typo and add Github link. Deploy Symfony application on Platform.sh.
weaverryan
added a commit
that referenced
this pull request
Dec 26, 2014
* 2.5: Update map.rst.inc Coding standard for PHP block. Fix coding standards #2 Use alphabetical order. Fix duplicated references. Add some references. Reference available services Better match Symfony documentation standard. Github => GitHub Fix typo and add Github link. Deploy Symfony application on Platform.sh.
weaverryan
added a commit
that referenced
this pull request
Dec 26, 2014
* 2.7: Update map.rst.inc Coding standard for PHP block. Fix coding standards #2 Use alphabetical order. Fix duplicated references. Add some references. Reference available services Better match Symfony documentation standard. Github => GitHub Fix typo and add Github link. Deploy Symfony application on Platform.sh.
ownsourcing
added a commit
to ownsourcing/symfony-docs
that referenced
this pull request
Dec 27, 2014
Radical proposal symfony#1: focus more specifically on creating pages in this document and put related (but not precisely 'dependent') discussions of directories, bundles and environments in a separate document called Structuring a Symfony App. Radical proposal symfony#2: take a heavy axe to The Book to eliminate every description, explanation and detail that's not at the highest possible altitude. Replace extraneous detail with links to the deeper information elsewhere, e.g. Best Practices. The ideal outcome would be DRYer docs and an easier to read and maintain The Book. Keep in mind that I'm a person with 'fresh eyes' vs one with a clear understanding of all objectives that must be met on this introductory page. I expect the best course is something between what I propose and what is currently there. That said, this revision takes an initial stab, so please feel free to use or ignore as you see fit.
weaverryan
added a commit
that referenced
this pull request
Jul 2, 2015
…pot, lyrixx, jdreesen, catchamonkey, gnugat, andreia, Arnaud Kleinpeter, willdurand, amitayh, nanocom, hrbonz, Pedro Gimenez, ubick, dirkaholic, bamarni, revollat, javiereguiluz) This PR was merged into the 2.3 branch. Discussion ---------- Integrate the "Create your own framework" tutorial This is a follow-up of the stalled #4455 PR. I've made most, but not all, of the changes proposed by reviewers. Specifically, I haven't changed the occurrences of `we` by `you`. The reason is that it's too much work. The original PR (submitted 8 months ago) died for that reason. So here it is my question: could we bypass the `we -> you` changes to merge it as soon as possible and fix that in further pull requests? Commits ------- bca9bae Updated doc references in the map.rst.inc file 108f109 Removed the numbers from the file names db9fa43 Made a lot of improvements suggested by reviewers 126bcef removed external references alignement 13a7170 made some changes to better integrate the tutorial into the current documentation bf9c871 fixed markup d44e4a2 added the new tutorial in the main index 73cd243 added the "Create your own framework" tutorial 409dba5 move things around f303b20 removed versions when adding Symfony component with Composer 55f5c12 updated the whole book (changes mainly related to Composer) 842e4d1 bug #25 Update part06.rst (revollat) a1336e0 Update part06.rst d802d42 updated code for Symfony 2.3 and made minor tweaks to the text f3c151c reworded slightly some sentence to convert the text from a series of articles to a book ca9d5d8 removed unused references 249b704 added missing links f6656e4 removed a note that is not relevant anymore 10e2732 added an index file c88c20a removed usage of the ClassLoader component in favor of Composer 2ddd8b9 merged branch bamarni/master (PR #24) 799e963 updated composer autoload path 91e46f6 removed the paragraph about CS as we now have standards 43e2e35 Merge pull request #23 from dirkaholic/master 6e01599 Fix little typo e8c19f7 Merge pull request #22 from ubick/master 8e7106d Fixed a typo in part02.rst (<php changed to <?php) c087780 merged branch amitayh/master (PR #17) 0d6beb1 merged branch pedrotgimenez/part11 (PR #21) 60617d7 Fixed one typo. aae0705 fixed CS 9533f9a merged branch hrbonz/part12_fixes (PR #19) d0ff8bc updated titles 400c087 add framework code as people would probably modify it following first code example (putting object creations in src/Simplex/Framework.php) de69a87 HttpKernel name can't be imported twice, if importing only Symfony\Component\HttpKernel\HttpKernel there will be problems later with HttpKernel subclasses (HttpKernel\Controller\ControllerResolver first and the others following). Could use 'use ... as ...' but I don't like it. 006b1e2 fixed markup 1f43dbf removed conflict merge 0941f47 merged branch Nanocom/master (PR #18) d907d46 Fixed typos db59374 fixed typo 11886e6 Merge branch 'master' of https://github.com/amitayh/Create-Your-Framework a8a2da0 Fixed part 12 typos 26bef8d Fixed typos 9bc692f added part 12 0b7581d merged branch willdurand/patch-1 (PR #16) 16b5b09 Fixed typo 831adec merged branch Nanocom/master (PR #15) a3f0b31 Corrected few typos ae1171a merged branch jdreesen/patch-3 (PR #10) 7a8e449 merged branch Nanocom/master (PR #14) 09969d9 added part 11 76e45f9 fixed markup fda9900 fixed pygments code name for json 02aab54 Corrected some english errors 8399581 moved the Context::fromRequest() code to the Framework class 111cac0 removed some use statement to be more consistent with previous parts a635d89 added part 10 bde64c8 made small tweaks 54e1b08 added a note in part 8 f00401d added part 8 5674395 renamed part as we now know that we are going to have more than 10 parts 479a85e fixed typos a0b1d42 merged branch gnugat/part7-typo-simple (PR #12) 9ffd186 [part7] Fixing typo: adding 'x' to Simple/Framework class 97743fb added part 7 fdb195c added part 6 0ce2a83 fixed some typos db0ad14 added part 5 bd3ca8e fixed extract() calls 7e8da09 fixed typo (closes #8) 1473eec fixed typo 358b4c8 added part 4 255577f went back to 2.1.* in composer.json files 3d46b1d merged branch jdreesen/patch-2 (PR #7) a18f827 fixed typos 18d376e updated composer.json 26be81c added part 3 4b39fc0 removed the path to composer.json as JSON doe snot support comments but some people copy/paste this line too 00e524e fixed ambiguity (closes #6) 6fe23c0 merged branch andreia/master (PR #5) 95edc8d Add missing colon to code-block 7e5cd41 merged branch gnugat/part2-typo-where-where (PR #4) 4c2e4f9 [part2] Fixinng typo: removing one of the double where ee67eee fixed markup 94db3cb merged branch catchamonkey/patch-1 (PR #3) a4f52d9 added links to projects fcaf268 Fixes grammatical error 2c79d42 added part 2 13ba87c added a tip about the Composer autoloader b93c118 merged branch jdreesen/patch-1 (PR #2) 269c6ce merged branch lyrixx/patch-1 (PR #1) b239305 Fixed typo 8ed4076 Fixed typo 99c5c08 added a LICENSE file 0ef3f8e added part 1
2 tasks
lsmith77
pushed a commit
to lsmith77/symfony-docs
that referenced
this pull request
May 3, 2018
added pledge to the enforcement team page
BooleanType
pushed a commit
to BooleanType/symfony-docs
that referenced
this pull request
Jan 10, 2021
HeahDude
added a commit
to HeahDude/symfony-docs
that referenced
this pull request
Aug 11, 2022
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey Fabien-
One commit, several very minor changes. I'll be reading over many of the chapters in the next week or two. If you'd like me to consolidate these small changes over several chapters into one branch & less pull request, let me know. Otherwise, I'll keep to very small branches with more pull requests.
Thanks