GitHub / ryanmcdermott/clean-code-javascript / commits
Clean Code concepts adapted for JavaScript
| SHA | Message | Author | Date | Stats |
|---|---|---|---|---|
| ccda22e6 | replace accidental tabs by spaces | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| d2c53647 | merge Merge branch 'master' into master |
Anton Paras <a****s@i****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 62afdb2b | merge Merge pull request #62 from vsemozhetbyt/typos |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 3ea61da2 | Refactor flags exmaple to be DRY | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 99b7116c | Refactor flags exmaple to be DRY |
Ryan McDermott <r****t@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 8d3f8da1 | merge Merge pull request #56 from sney2002/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| ec8fd242 | Remove subsection regarding legal comments in source files | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| 41b9ee43 | Remove subsection regarding legal comments in source files |
Ryan McDermott <r****t@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| ac47e823 | Remove ES6 constants subsection | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| c0545983 | Remove ES6 constants subsection |
Ryan McDermott <r****t@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| d333e12d | Fix method chaining subsection typo | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| d8c12c0c | Fix method chaining subsection typo |
Ryan McDermott <r****t@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| d28b1e3c | Remove quip about JavaScript and Haskell being like beer types/brands | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| fc0f5c73 | Remove quip about JavaScript and Haskell being like beer types/brands |
Ryan McDermott <r****t@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| ce9f41fe | merge Merge branch 'master' into patch-1 |
Christian Schlensker <c****n@c****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| d7cc3704 | merge Merge branch 'master' into simplify |
Vse Mozhet Byt <v****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 2dc28e2d | merge Merge branch 'master' into typos |
Vse Mozhet Byt <v****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| d35896ba | merge Merge pull request #53 from DJTB/patch-2 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| e88e5366 | merge Merge pull request #66 from darahak/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 04c29e4c | merge Merge pull request #54 from sney2002/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 4a2e3c4e | merge Merge pull request #58 from danielstjules/danielstjules-patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| e9b473f8 | merge Merge pull request #60 from rodu/minor-naming-improvement |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 576f4ef1 | merge Merge pull request #69 from vsemozhetbyt/indentation |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 62f0b70c | merge Merge pull request #51 from schoenwaldnils/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| f4467831 | refactor(explanatory-variables): add input example & remove first group | Sébastien Elet <s****n@e****r> | over 9 years ago | |
| 9f0289bf | fix little typo in arrow function argument's remove duplicate code section | Rafael Poveda <r****o@g****m> | over 9 years ago | |
| 509260ba | fix indentation in a code example |
Vse Mozhet Byt <v****t@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 37344ac7 | fix indentation in a code example | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| 63ca1861 | merge Merge branch 'master' of github.com:ryanmcdermott/clean-code-javascript | Nils Schönwald <n****s@s****a> | over 9 years ago | |
| 5da0302c | Fix typo |
Nils Schönwald <n****s@s****a>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 2200e40a | Fix typo | Nils Schönwald <n****s@s****a> | over 9 years ago | |
| 54796aad | Mention async functions as part of ES2017 instead of ES7 (ES2016) + switch to... |
Dara Hak <d****k@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 75998596 | Mention async functions as part of ES2017 instead of ES7 (ES2016) + switch to... | Dara Hak <d****k@g****m> | over 9 years ago | |
| 9fe05688 | fixed few examples by adding missing semicolons | Hemant Pawar <h****t@h****m> | over 9 years ago | |
| 14e83d5c | use functional Array methods | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| 7035841c | remove default constructors | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| 465f3bfd | use object shorthand notation | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| 77102116 | fix some typos in the README.md | Vse Mozhet Byt <v****t@g****m> | over 9 years ago | |
| 68300c2c | fix some typos in the README.md |
Vse Mozhet Byt <v****t@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 5d882d92 | Improvement on email client function | rodu <g****5@g****m> | over 9 years ago | |
| 9272fa29 | Improvement on email client function |
rodu <g****5@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| deef735a | Add mention of jsinspect for duplicate code detection |
Daniel St. Jules <d****s@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| d6e52d56 | Add mention of jsinspect for duplicate code detection |
Daniel St. Jules <d****s@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 0c12d9b7 | use this to access object methods |
Jhonatan Salguero Villa <s****2@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 8135d47c | use this to access object methods |
Jhonatan Salguero Villa <s****2@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 3adcc2c4 | return true on save method to continue chaining |
Jhonatan Salguero Villa <s****2@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 2da4d4b4 | return true on save method to continue chaining |
Jhonatan Salguero Villa <s****2@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| b4935260 | fix output in Object.assign() example |
Duncan Bay <D****B@u****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 45f40fac | fix output in Object.assign() example |
Duncan Bay <D****B@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| ddd01819 | merge Merge pull request #43 from jordalgo/jordalgo/errorHandling |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 8be1793b | Change Bad/Better/Best to Bad/Bad/Good |
Anton Paras <a****s@i****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 2f1142fd | merge Merge pull request #39 from acidio/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 7876a64a | merge Merge pull request #46 from stewx/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 17b01a2b | Change var and let to const where necessary | Nils Schönwald <n****s@s****a> | over 9 years ago | |
| 992b1b00 | Change var and let to const where necessary |
Nils Schönwald <n****s@s****a>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| fd293bc9 | Improve example "Function names should say what they do" | Tarabass <p****d@l****m> | over 9 years ago | |
| e35533fd | Correct spelling of Cessna airplane |
Stewart Rand <s****d@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 243a5451 | Correct spelling of Cessna airplane |
Stewart Rand <s****d@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| a74ba88c | Add: Error Handling Section |
Jordalgo <j****o@s****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 100628cd | Add: Error Handling Section | Jordalgo <j****o@s****m> | over 9 years ago | |
| fe036256 | Adjusted the commentthat show the result |
Acidio Alan <a****o@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 64b771fa | Adjusted the commentthat show the result | Acidio Alan <a****o@g****m> | over 9 years ago | |
| 19b1f36d | feat(explanatory-variables): use destructuring | Sébastien Elet <s****n@e****r> | over 9 years ago | |
| 10bece91 | Use more plausible const example |
Christian Schlensker <c****n@c****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 2837f0fe | merge Merge pull request #35 from ChrisNikkel/use_const |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 08dd958d | use const in rule "Use explanatory variables" |
Christopher Nikkel <c****s@i****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 9fc2093a | use const in rule "Use explanatory variables" |
Christopher Nikkel <c****s@i****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| f0278efd | Use const instead of var |
Christian Schlensker <c****n@c****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| dfb9544d | Rephrase default value advice |
Anton Paras <a****s@i****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| a22d1d90 | merge Merge pull request #32 from acparas/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| fa5f27d7 | Fix semantic mistake |
Anton Paras <a****s@i****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| de317862 | Fix semantic mistake |
Anton Paras <a****s@i****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 11b72718 | merge Merge pull request #27 from connec/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 52a92517 | merge Merge pull request #29 from jordalgo/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 6873385d | Update: promise catch logic |
Jordan Rome <j****n@j****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| e9044457 | Update: promise catch logic |
Jordan Rome <j****n@j****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 55e612d6 | merge Merge pull request #28 from tonytran1/master |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 39c75cf4 | Use constants when variable values do not change |
tonytran1 <t****e@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| f090f77a | Use constants when variable values do not change | tonytran1 <t****e@g****m> | over 9 years ago | |
| c51445ef | merge Merge pull request #26 from iheanyi/patch-2 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 3f973412 | Remove redundant object from Object.assign example |
Chris Connelly <c****s@c****k>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 17a5f24a | Remove redundant object from Object.assign example |
Chris Connelly <c****s@c****k>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 337fa7e9 | Update README.md |
Iheanyi Ekechukwu <i****u@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| feaa4e16 | Update README.md |
Iheanyi Ekechukwu <i****u@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| a35666a4 | Fix Object.assign example |
Ryan McDermott <r****t@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| d00ca4c7 | Fix Object.assign example | Ryan McDermott <r****t@g****m> | over 9 years ago | |
| bd8bd1e2 | merge Merge pull request #20 from bfred-it/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 23887983 | merge Merge pull request #21 from Chaosbohne/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 5cdf7a80 | merge Merge pull request #22 from floriangosse/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 5d3bd8ae | merge Merge pull request #24 from hennelh/patch-1 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| e8023809 | Update README.md |
Henry Harrison <h****h@H****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 6b550875 | Update README.md |
Henry Harrison <h****h@H****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| b1ef827d | fix(avoid-type-checking-part-1): fix variable name |
Florian Goße <f****o@e****e>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 28740cc5 | fix(avoid-type-checking-part-1): fix variable name |
Florian Goße <f****o@e****e>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 08fe61e8 | Some minor property missmatch |
Rico Ruszewski <C****e@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| 70aa57fa | Some minor property missmatch |
Rico Ruszewski <C****e@u****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| a037f0e0 | Add missing function call to "no side effects" |
Federico Brigante <b****t@u****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| dd185ca6 | Add missing function call to "no side effects" |
Federico Brigante <b****t@u****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago | |
| 2564bae2 | merge Merge pull request #19 from gsklee/patch-2 |
Ryan McDermott <r****t@g****m>
Committed by: GitHub <n****y@g****m> |
over 9 years ago | |
| acfc98fc | Update README.md |
G. Kay Lee <b****b@g****m>
Committed by: Francesco Colonna <f****o@g****m> |
over 9 years ago |