Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Minor fixes #4

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Minor fixes #4

wants to merge 5 commits into from

Conversation

marcolussetti
Copy link

I was just taking a peek at the website and noticed a few minor things that could be cleared up :)

  • Fixed a typo in the membership section
  • Some of the links to the forum we broken, and figured I'd switch the rest to https while I was at it
  • Removed Google Plus as that service has been discontinued
  • I noticed a whole bunch of inconsistencies in the indentation (mix of tab and spaces, and not always with the same level) which were making it challenging to edit, so I figured I'd try to clean it up a bit. It looked like you guys were going for the 2 spaces JS convention so I tried to standardize on that

@marcolussetti
Fix minor typo
929226c
@marcolussetti
Fix dead community links, & switch to https
89bd728
@marcolussetti
Remove Google Plus (discontinued by Google)
5d374ee
@marcolussetti
Clean up indent
eb3874a 
I was struggling a bit to edit the file as it had a mix of tab and
spaces, with what looked like a 2 space indent objective.

Cleaned up the indentation and standardized on 2 space indent. Obviously
no attachment to it, was just try to standardize on what looked like the
original intent
@marcolussetti
Missed a bit of indents
58b8cf2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@marcolussetti