-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: tidy up document #133
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
appears to be lots of white space in update??
We can flatten it if you would like? see the tidyconfig.txt |
Let me know what you are most comfortable with... I prefer the indenting, as I find table nesting quite challenging to edit manually. We can also boost the line length or whatever you like. |
@marcoscaceres I am fine with it being flattened |
ok, flat for ya :) what about line length? Keep 80? |
Also... about the indenting.... I'm worried that without it, external contributors might have a hard time making changes (specially people like me who are dyslexic). I'd really struggle with flat code, so it might be a service to the community to indent the code. |
@marcoscaceres I am also fine with indenting :-) |
Closes #132
Preview | Diff