Like many developers I have a certain style to all my code. I have decided to enumerate them here along with an accompanying eslint rule or accessability related reasoning for its use.
this can be installed via NPM
npm install eslint-config-arri
you can then extend this in your eslint configuration.
{
"extends": "arri"
}
1.1 Indentation eslint:indent
All indentation will be done with tabs.
This is to allow for greater accessability for persons with limited sightedness. Read more about it in Nobody talks about the real reason to use Tabs over Spaces.
1.2 Padding: let your code breath eslint:padding-line-between-statements
When writing code make sure that it does not give the impression of cramping, there should be spaces between logical sections of code.
Persons with attention deficit disorders and ASD conditions, have difficulty reading "wall of text" style documentation and code. Padding the code out will help.
Place an empty line between your section of variable declarations and the next non-declaration statement in your code. This separates the code out and makes it more readable.
✔️Good
const numbers = [1,2,3,4,5,6];
for(let number of numbers) {
//...
}
const a = 1;
const b = 2;
if(number !== a || number !== b) {
//...
}
❌Bad
const numbers = [1,2,3,4,5,6];
for(let number of numbers) {
//...
}
const a = 1;
const b = 2;
if(number !== a || number !== b) {
//...
}
Multiline blocks, expressions, declarations and control statements must have a space before and after them.
✔️Good
runMe();
callTheFunctionWithLotsOfArgs({
field: value,
field2: value,
});
runMeToo();
runMeThree();
❌Bad
runMe();
callTheFunctionWithLotsOfArgs({
field: value,
field2: value,
});
runMeToo();
runMeThree();
When declaring a class or function there must be space before. The one exception to this is declarations of variables in the top of a function block.
✔️Good
class MyClass {
constructor() {
//...
}
}
function() {
doStuff();
}
❌Bad
class MyClass {
constructor() {
//...
}
}
function() {
doStuff();
}
2.1 Curly should be used over single line statements eslint:curly
All blocks and block-like statements must have curly brackets around the content.
✔️Good
if(isTrue) {
doSomething();
}
while(isTrue) {
doSomething();
}
do {
doSomething();
} while(isTrue);
❌Bad
if(isTrue) doSomething();
while(isTrue) doSomething();
do doSomething() while(isTrue);
This makes for easier reading when diffing files. The condition is on its own line and the actions are as well. If more code needs to be added to the block it will just show up as an add instead of a modified line.