GitHub / ryanmcdermott/clean-code-javascript / commits
Clean Code concepts adapted for JavaScript
| SHA | Message | Author | Date | Stats |
|---|---|---|---|---|
| daf2d6c7 | Add caveats |
Kostas Bariotis <k****r@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 5d209daf | Typo and rewording |
Kostas Bariotis <k****r@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 00a50681 | merge Merge pull request #144 from vsemozhetbyt/typo |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| ca1cd965 | remove duplicate word in 'Remove duplicate code' :) | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| 3b42f996 | merge Merge pull request #141 from vsemozhetbyt/nits |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| ba3e34fc | fix some code style nits in examples | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| b1b37eb2 | Make meaningful variable example even clearer | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 27219805 | merge Merge pull request #140 from krzysztof-grzybek/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| aa71b306 | modify open/closed principle | krzysztof-grzybek <k****9@g****m> | over 9 years ago | |
| 3364af5f | Remove stark warning against duplicate code, let's just strongly suggest to a... | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 84f8ea6f | Simplify "Use searchable names" example (#121) |
Crisoforo Gaspar Hernández <h****o@c****m>
Committed by: Ryan McDermott <r****t@g****m> |
over 9 years ago | |
| aa069b37 | merge Merge pull request #128 from ianstormtaylor/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| e94afcf3 | Change literal for a constant | Marc Aragonès <m****s@g****m> | over 9 years ago | |
| dd47a0d0 | merge Merge pull request #134 from ejribeiro/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 9becf1c0 | Fix lookupManager typo | ribeiroje <e****o@b****t> | over 9 years ago | |
| 7ec1fe7a | merge Merge pull request #83 from iamken1204/fill-missing-commas |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| e550f1ea | merge merge origin | Kettan <i****4@g****m> | over 9 years ago | |
| ab48c4c8 | merge Merge pull request #118 from vasanthk/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| a09a6968 | merge Merge pull request #124 from vsemozhetbyt/diff |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| d97805d6 | Remove Law of Demeter | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 6774f750 | fix async/await example |
Ian Storm Taylor <i****n@i****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 54aed622 | make bad and old examples more comparable | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| e8812779 | Add Side Effects pt.2 |
Kostas Bariotis <k****r@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 30e7a39b | Fix naming for bad code as well |
Vasa <v****k@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| cd508897 | merge Merge pull request #65 from vsemozhetbyt/simplify |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 08601465 | merge Merge pull request #114 from vsemozhetbyt/new-clean |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 783f7c58 | merge Merge branch 'master' into new-clean |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| e7ef70e3 | merge Merge pull request #120 from JakoD/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 05089ba5 | "Use searchable names" and correct unit of time |
Jako Danar <J****D@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 7761d249 | Fix erroneous line break |
Vasa <v****k@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 887c779b | Rename state to zipCode to avoid confusion |
Vasa <v****k@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 6fcbecea | fix new instances of previously fixed cases | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| 383dd491 | merge Merge pull request #64 from HemantPawar/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| e4100a30 | merge Merge pull request #111 from irazasyed/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 34abb0bb | Fix Typo |
Syed Irfaq R. ⚡️ <i****d@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 0b3aba1a | merge Merge branch 'master' into master |
Hemant Pawar <h****t@h****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 50850a93 | Fix unorganized list in Objects section | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 50e5c1b0 | merge Merge pull request #33 from wordofchristian/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| cea298d2 | merge Merge pull request #37 from SebastienElet/feat/use-destructuring |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 67942527 | merge Merge pull request #34 from acparas/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| f93513e6 | Fix argument ordering | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| bf749576 | merge Merge pull request #47 from Tarabass/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| f350f2be | merge Merge branch 'master' into master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 0c4fa28c | merge Merge pull request #81 from vsemozhetbyt/arrow-parens |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 99bee74a | merge Merge pull request #82 from vsemozhetbyt/prefer-template |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 87d6d3bb | merge Merge pull request #95 from vsemozhetbyt/anon-functions |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 7fe566c5 | Fix extra good example in getters and setters | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 0951bc27 | merge Merge pull request #101 from kurtommy/fix/getter-setter |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| ce4315fd | function name change |
Tommaso Rossi <k****y@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| fda83577 | Update README.md |
Tommaso Rossi <k****y@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| eaa3477d | Lower "bad example" count |
Anton Paras <a****s@i****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| aec7b4c4 | fix misspelled property name |
Vse Mozhet Byt <v****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| f5e59ee9 | merge Merge pull request #90 from vsemozhetbyt/single-quotes |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 3bf55e07 | merge Merge pull request #93 from vsemozhetbyt/brace-style |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 8f7c8cee | merge Merge pull request #102 from othree/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 58ace722 | merge Merge pull request #105 from checkaayush/fix-typo |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| b326ee5a | Fix consecutive occurrence of 'have' | Aayush Sarva <c****h@g****m> | over 9 years ago | |
| f70ac099 | merge Merge branch 'master' into master |
Hemant Pawar <h****t@h****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 23e0da33 | Use `console.error` to deal with error | othree <o****e@g****m> | over 9 years ago | |
| e56b69e4 | Added real getter and setter |
Tommaso Rossi <k****y@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| d7b8cc1e | Update README.md |
Tommaso Rossi <k****y@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 1f1cd46a | merge Merge branch 'master' into anon-functions |
Vse Mozhet Byt <v****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 44c32412 | merge Merge branch 'master' into brace-style |
Vse Mozhet Byt <v****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 0b558727 | merge Merge branch 'master' into single-quotes |
Vse Mozhet Byt <v****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| fcfab4e4 | merge Merge pull request #91 from vsemozhetbyt/eqeqeq |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| c6137725 | merge Merge pull request #88 from vsemozhetbyt/class-methods-use-this |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| d9712543 | merge Merge pull request #92 from vsemozhetbyt/break |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 847e340f | merge Merge pull request #94 from vsemozhetbyt/operator-assignment |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 3f94eb22 | merge Merge pull request #96 from vsemozhetbyt/no-use-before-define |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 2d59de73 | merge Merge pull request #97 from vsemozhetbyt/no-shadow |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| bfa3eb30 | merge Merge pull request #98 from danielstjules/magic-numbers |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| ac12cb81 | merge Merge branch 'master' into master |
Anton Paras <a****s@i****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 63b46b58 | Mention buddy.js and ESLint for magic numbers | Daniel St. Jules <d****s@g****m> | over 9 years ago | |
| bd5d9ef4 | Fix package name, s/jsinpect/jsinspect | Daniel St. Jules <d****s@g****m> | over 9 years ago | |
| 25791b03 | fix variable shadowing | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| b70664e2 | reorder class declarations | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| b11adc4f | address anonymous functions | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| d6b1a712 | use assignment operators | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| df660d3b | fix a brace style inconsistency | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| 82618ee1 | add `break` statement to prevent erroneous fall-through | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| 83a565ec | replace == with === | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| c92b7928 | use single quotes consistently | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| f3f62618 | correct using of `this` in examples | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| 594dd628 | merge Merge branch 'master' into arrow-parens |
Vse Mozhet Byt <v****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| d279012d | merge Merge branch 'master' into simplify |
Vse Mozhet Byt <v****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 907617b8 | merge Merge pull request #87 from vsemozhetbyt/const-let |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 8e469dba | merge Merge pull request #86 from Asheboy/less-fewer-grammar |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| d59f32ed | merge Merge pull request #76 from raerpo/fix-typo |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 3369c524 | merge Merge pull request #80 from vsemozhetbyt/comment-out-ellipses |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| b3b40209 | fix new cases with const and let | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| 5c2f16a7 | merge Merge pull request #79 from vsemozhetbyt/tabs-to-spaces |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 82f9d39b | merge Merge pull request #84 from posva/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 86036b50 | Corrects usages of less for fewer |
Asheboy <a****S@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| c3e754db | merge Merge branch 'master' into fix-typo |
Rafael E. Poveda <r****o@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| d2d70517 | Fix indentation in code |
Eduardo San Martin Morote <p****a@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 748ba65f | merge Merge branch 'master' into master |
Hemant Pawar <h****t@h****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| fadfb57b | Fill missing commas | Kettan <i****4@g****m> | over 9 years ago | |
| 73817c71 | replace string concatenation by template literals | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| eb264270 | parenthesize arrow function arguments consistently | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| f9fb0784 | comment out all the ellipses consistently | Vse Mozhet Byt <v****t@g****m> | over 9 years ago |