Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
Accelerator
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Search
/
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
Uh oh!
There was an error while loading.
Please reload this page
.
javascript-basket
/
clean-code-javascript
Public
forked from
ryanmcdermott/clean-code-javascript
Notifications
You must be signed in to change notification settings
Fork
0
Star
0
Code
Pull requests
0
Actions
Projects
Wiki
Security and quality
0
Insights
Additional navigation options
Code
Pull requests
Actions
Projects
Wiki
Security and quality
Insights
Commits
Branch selector
master
User selector
All users
Datepicker
All time
Commit history
Commits on Jan 5, 2017
Merge pull request #18 from gsklee/patch-1
Show description for 4736752
ryanmcdermott
authored
4736752
View commit details
Copy full SHA for 4736752
Browse repository at this point
Update README.md
Show description for 4155a2b
G. Kay Lee
authored
4155a2b
View commit details
Copy full SHA for 4155a2b
Browse repository at this point
Merge pull request #17 from jsgao0/master
Show description for 37c236c
ryanmcdermott
authored
37c236c
View commit details
Copy full SHA for 37c236c
Browse repository at this point
Update README.md
Show description for a79076f
jsgao0
authored
a79076f
View commit details
Copy full SHA for a79076f
Browse repository at this point
Merge pull request #16 from jsgao0/master
Show description for c0cd8f8
ryanmcdermott
authored
c0cd8f8
View commit details
Copy full SHA for c0cd8f8
Browse repository at this point
Update README.md
Show description for b5c7100
jsgao0
authored
b5c7100
View commit details
Copy full SHA for b5c7100
Browse repository at this point
Merge pull request #8 from alaycock/patch-1
Show description for 03bcebd
ryanmcdermott
authored
03bcebd
View commit details
Copy full SHA for 03bcebd
Browse repository at this point
Merge pull request #10 from Yahkob/patch-2
Show description for 667ff35
ryanmcdermott
authored
667ff35
View commit details
Copy full SHA for 667ff35
Browse repository at this point
Merge pull request #11 from Yahkob/patch-4
Show description for dcbca43
ryanmcdermott
authored
dcbca43
View commit details
Copy full SHA for dcbca43
Browse repository at this point
Merge pull request #6 from chiarafsc/master
Show description for e560b13
ryanmcdermott
authored
e560b13
View commit details
Copy full SHA for e560b13
Browse repository at this point
Merge pull request #3 from mariotacke/fix-spelling-errors
Show description for 20a8c6c
ryanmcdermott
authored
20a8c6c
View commit details
Copy full SHA for 20a8c6c
Browse repository at this point
Commits on Jan 4, 2017
add missing commas in objects
Yahkob
authored
ce30644
View commit details
Copy full SHA for ce30644
Browse repository at this point
Use explanatory variables bad and good examples should match ([1] + [2] instead of [0])
Yahkob
authored
071dfdc
View commit details
Copy full SHA for 071dfdc
Browse repository at this point
Merge pull request #1 from alaycock/master
Show description for b49c571
alaycock
authored
b49c571
View commit details
Copy full SHA for b49c571
Browse repository at this point
Add missing function argument in LSP section
alaycock
authored
95ad6fb
View commit details
Copy full SHA for 95ad6fb
Browse repository at this point
Add getArea method where it is necessary.
Show description for cc8c767
alaycock
authored
cc8c767
View commit details
Copy full SHA for cc8c767
Browse repository at this point
Remove duplicate code "good" example - Remove unnecessary, misleading statement
chiarafsc
committed
0a1fc61
View commit details
Copy full SHA for 0a1fc61
Browse repository at this point
(docs) Correct spelling errors and remove unfinished sentence.
mariotacke
committed
a3e6e97
View commit details
Copy full SHA for a3e6e97
Browse repository at this point
Merge pull request #1 from andrejewski/patch-1
Show description for bc1ae6a
ryanmcdermott
authored
bc1ae6a
View commit details
Copy full SHA for bc1ae6a
Browse repository at this point
fix "do one thing" code example
Show description for 0661e52
andrejewski
authored
0661e52
View commit details
Copy full SHA for 0661e52
Browse repository at this point
Commits on Jan 3, 2017
Functions should be at one level of abstraction
ryanmcdermott
committed
5530480
View commit details
Copy full SHA for 5530480
Browse repository at this point
Drive home pont of limiting function arguments
ryanmcdermott
committed
b8b382d
View commit details
Copy full SHA for b8b382d
Browse repository at this point
Remove subsection anchors, favor automatic marking
ryanmcdermott
committed
d783297
View commit details
Copy full SHA for d783297
Browse repository at this point
Attempt another GHFM link
ryanmcdermott
committed
4440e35
View commit details
Copy full SHA for 4440e35
Browse repository at this point
Add first subsection marker
ryanmcdermott
committed
8c13a41
View commit details
Copy full SHA for 8c13a41
Browse repository at this point
Commits on Jan 2, 2017
Single concept per test
ryanmcdermott
committed
f871283
View commit details
Copy full SHA for f871283
Browse repository at this point
Encapsulate conditionals
ryanmcdermott
committed
8f4a8d2
View commit details
Copy full SHA for 8f4a8d2
Browse repository at this point
Avoid negative conditionals
ryanmcdermott
committed
0382051
View commit details
Copy full SHA for 0382051
Browse repository at this point
Function names should say what they do
ryanmcdermott
committed
e7ca5c9
View commit details
Copy full SHA for e7ca5c9
Browse repository at this point
Remove dead code
ryanmcdermott
committed
f83f4e4
View commit details
Copy full SHA for f83f4e4
Browse repository at this point
Avoid positional markers
ryanmcdermott
committed
ea1735d
View commit details
Copy full SHA for ea1735d
Browse repository at this point
Use explanatory variables
ryanmcdermott
committed
c080300
View commit details
Copy full SHA for c080300
Browse repository at this point
Add proper introduction
ryanmcdermott
committed
85bfa0c
View commit details
Copy full SHA for 85bfa0c
Browse repository at this point
Add coverage tool note
ryanmcdermott
committed
d02640b
View commit details
Copy full SHA for d02640b
Browse repository at this point
Add testing section
ryanmcdermott
committed
887ee23
View commit details
Copy full SHA for 887ee23
Browse repository at this point
Previous
Next
You can’t perform that action at this time.