GitHub / ryanmcdermott/clean-code-javascript / commits
Clean Code concepts adapted for JavaScript
| SHA | Message | Author | Date | Stats |
|---|---|---|---|---|
| 72fd417a | merge Merge pull request #232 from ngdo-pro/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
almost 9 years ago | |
| dda95822 | Update README.md |
Nicolas Gomes De Oliveira <n****7@g****m>
Committed by: GitHub <n****y@g****m> |
almost 9 years ago | |
| b3e1359f | merge Merge pull request #229 from scraggo/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
almost 9 years ago | |
| 672434a4 | minor grammar edit |
Dave C <d****8@g****m>
Committed by: GitHub <n****y@g****m> |
almost 9 years ago | |
| 6b90694a | merge Merge pull request #228 from vokar97/vokar97/replace-map-with-destructuring |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
almost 9 years ago | |
| 1d7362d9 | Replace map method with object destructuring |
Vladimir Karamyshev <v****7@g****m>
Committed by: GitHub <n****y@g****m> |
almost 9 years ago | |
| ea0e4a85 | merge Merge pull request #227 from open-node/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
about 9 years ago | |
| 0969000e | merge Merge pull request #226 from snirad/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
about 9 years ago | |
| 177b636b | Update README.md |
snirad <s****d@u****m>
Committed by: GitHub <n****y@g****m> |
about 9 years ago | |
| 2ae57ca9 | 'data' is never reassigned. Use 'const' instead. | Redstone Zhao <1****0@q****m> | about 9 years ago | |
| 288bf218 | merge Merge pull request #225 from LayZeeDK/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
about 9 years ago | |
| fdbd9078 | Object style consistency |
Lars Gyrup Brink Nielsen <l****k@g****m>
Committed by: GitHub <n****y@g****m> |
about 9 years ago | |
| 025f7f93 | merge Merge pull request #222 from filipetedim/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
about 9 years ago | |
| e12f62f1 | Fixed duplicate code |
filipetedim <f****m@g****m>
Committed by: GitHub <n****y@g****m> |
about 9 years ago | |
| 6e7b67bd | merge Merge pull request #220 from GrzegorzZajac000/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
about 9 years ago | |
| 465c3800 | Pass make, model and color in constructor |
Grzegorz Zając <g****0@g****m>
Committed by: GitHub <n****y@g****m> |
about 9 years ago | |
| eae25750 | Update README.md |
Grzegorz Zając <g****0@g****m>
Committed by: GitHub <n****y@g****m> |
about 9 years ago | |
| 51683128 | Just Add an Indonesia Language Link (#219) |
Andi R. Hermawan <a****1@m****d>
Committed by: Ryan McDermott <r****t@g****m> |
about 9 years ago | |
| f727215a | merge Merge pull request #217 from greg-dev/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| ccd53bfb | Add Polish translation | Greg Dev <g****v@g****m> | over 9 years ago | |
| 3a3bb901 | merge Merge pull request #206 from rouzbeh84/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 0a8199a9 | merge Merge pull request #216 from Neatoro/fix-duplicate-code |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| f5085c92 | Change indentation to spaces |
Moritz Schramm <N****o@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 3b350e1d | Fix indentation |
Moritz Schramm <N****o@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 63db5914 | Fix code example for duplicate code | Schramm, Moritz <m****m@s****m> | over 9 years ago | |
| 578f35c6 | merge Merge pull request #215 from elliothimmelfarb/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| db8790fa | fixed a grammatical typo |
Elliot Himmelfarb <e****b@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 20cd8441 | Update README.md |
rouzbeh84 <r****4@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 8e89be37 | merge Merge pull request #210 from mitsuruog/ja |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| d70698c2 | Add Japanese translation link | Mitsuru Ogawa <m****p@g****m> | over 9 years ago | |
| 94076b02 | modify function name for `isActiveClient` |
rouzbeh84 <r****4@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 43c9f7d7 | merge Merge pull request #207 from kantuni/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| a995cc49 | Mistake in the Set default objects with Object.assign |
Henrikh Kantuni <h****i@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 59bc2d01 | Update README.md |
rouzbeh84 <r****4@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 2ff9c4f2 | Add 3 Rs of Software Architecture link | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 3720dc54 | merge Merge pull request #202 from feimosi/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| eb9d05bc | Fix broken link |
Marek Grzybek <f****i@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 70cb4d23 | Add beginor's Chinese translation | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| d0ecda5f | merge Merge pull request #197 from piotrwitek/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| f868d6fd | merge Merge pull request #198 from andersontr15/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 915ce719 | update link |
Theodore Rollin Anderson <t****3@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| f6c1f729 | fix repo link |
Theodore Rollin Anderson <t****3@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 72b4a293 | working on spanish version of your repo |
Theodore Rollin Anderson <t****3@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 6992d6e4 | fixed mismatch between bad-good example |
Piotr Witek <p****k@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 7af69850 | merge Merge pull request #196 from rohmanhm/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 7177853a | fix angularjs typing | Muhammad Habib Rohman <m****n@l****m> | over 9 years ago | |
| fce69199 | merge Merge pull request #193 from imsobear/fixed-code-style |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 65e10142 | merge Merge pull request #192 from timjacobi/require-import |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 47330269 | fix: code style | liuxiong.lx <l****x@a****m> | over 9 years ago | |
| 3a25ed2f | Break SOLID principles out into their own section | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 3108a3b2 | use ES6 style imports instead of require | timjacobi <d****v@t****m> | over 9 years ago | |
| dce1bcd0 | merge Merge pull request #187 from maksugr/add-russian-translation |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 323682ff | merge Merge branch 'master' into add-russian-translation |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| daa074af | merge Merge pull request #186 from hienvd/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| d5ecffeb | merge Merge pull request #188 from maksugr/typo-fix-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| a59d78b4 | Fixes some typo issues | Roman Ponomarev <m****r@g****m> | over 9 years ago | |
| 0358273e | Adds second and actual russian translation | Roman Ponomarev <m****r@g****m> | over 9 years ago | |
| 60a5a148 | Add vietnamese translation | hienvd <d****g@g****m> | over 9 years ago | |
| b82b63bf | merge Merge pull request #183 from DavidVujic/getters-setters |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 69aac044 | getters and setters as functions, also without the get/set keywords | davidvujic <d****d@v****e> | over 9 years ago | |
| ac6a51d6 | merge Merge pull request #182 from DavidVujic/liskov-substitution |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 90e9b243 | fix: typo | davidvujic <d****d@v****e> | over 9 years ago | |
| a624d544 | refactor: Liskov Substitution - width/height/length as constructor parameters | davidvujic <d****d@v****e> | over 9 years ago | |
| 47dbe9dc | Explicit function name (#177) |
Anna <a****c@m****m>
Committed by: Ryan McDermott <r****t@g****m> |
over 9 years ago | |
| 49df233e | merge Merge pull request #176 from hueitan/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| bfd763a0 | Update the example from "Single concept per test" |
Huei Tan <h****0@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 471371e4 | Add note about default arguments | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 348760e0 | merge Merge pull request #173 from DavidVujic/suggestions |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 37d929c5 | refactor: inline function | davidvujic <d****d@v****e> | over 9 years ago | |
| beaaa370 | Object instead of Class | davidvujic <d****d@v****e> | over 9 years ago | |
| eec04a7b | merge Merge pull request #171 from travellingprog/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 166d7d41 | merge Merge pull request #172 from BoryaMogila/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 8f37510b | Add russian translation |
BoryaMogila <b****g@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| dd791e97 | README.md: change 'peddle' to 'pedal' |
Erick Cardenas-Mendez <e****z@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| b436426c | merge Merge pull request #166 from MarkWaldron/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 59d41827 | Fix class method call in getter/setter example | MarkWaldron <m****n@g****m> | over 9 years ago | |
| a8c20461 | Improve language of function arguments subsection | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 7c709688 | merge Merge pull request #165 from bali182/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 209f944c | Changed Function arguments section to include destructuring | Édes Balázs <e****s@g****m> | over 9 years ago | |
| 35f5c5c8 | merge Merge pull request #137 from MarcAragones/apply_searchable_name |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 62dfd203 | Fix some **Good**: to **Good:** (#145) |
Eduardo Sganzerla <e****a@g****m>
Committed by: Ryan McDermott <r****t@g****m> |
over 9 years ago | |
| 85029c42 | merge Merge pull request #157 from qkraudghgh/translate |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| b22ab6bc | Add korean translation | myoungho.Pak <q****h@n****m> | over 9 years ago | |
| d41598d1 | Add German translation | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| e075d8d1 | merge Merge pull request #156 from fesnt/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 4f6d1418 | Add translation section | Felipe Santos <f****l@g****m> | over 9 years ago | |
| 4d082341 | Fix minor markdown error | Felipe Santos <f****l@g****m> | over 9 years ago | |
| f0a34877 | merge Merge pull request #153 from vsemozhetbyt/naming |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 35a0cf8f | fix confusing example | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| ab29de61 | merge Merge pull request #152 from Gregoirevda/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 55ec74a1 | Immutable error |
Van der Auwermeulen Grégoire <g****a@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| ff6fdc14 | Fix side effect part 2 list | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 68447308 | Expand side effect part 2 discussion | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 5e581f9a | merge Merge pull request #123 from kbariotis/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 7dabe473 | merge Merge pull request #149 from Endlessline/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 89a0725d | merge Merge pull request #147 from LiranBri/composition-over-inheritance |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 17512c31 | Update README.md |
Collin <c****e@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| fae51115 | Update code example |
Kostas Bariotis <k****r@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| ec8dbd19 | fix is-a order in Prefer composition over inheritance description | Liran Brimer <l****r@v****m> | over 9 years ago | |
| a4718d4d | Change example |
Kostas Bariotis <k****r@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago |