Leaderboard Route

[RachitKeertiDas]

Add a Route for Leaderboard. Handle GET requests from the user to serve him the leaderboard in the form of JSON.

Also, the helper ServeJSON should simply return incase there is an error is encoding into JSON and should not write a malformed JSON.

[AdaephonBen]

Nice catch! Don't handle the error by just returning; log a warning using the appropriate function in package logs. Also, run the following command inside the server directory once you're done: gofmt -s -w . This will format all the code you've written.

[RachitKeertiDas]

Thanks for the gofmt tip! I have added the required changes. To log errors, I have imported the "log" package in helpers.go.

I have used the standard log package as this package has been used in handlers.go. However, the logs package has not been used anywhere in the routes directory.

[npalladium]

Notes:

• Give concise but useful Git Commit messages. Yours are useful. You can make them more concise. Also, our style for Git messages is imperative.
• Finally, nothing but a working product gets merged into master. Always use develop or a proper development branch in the future.
• We follow a modified version of GitLab Flow which in itself is a modified version of GitHub flow. The main difference right now is the use of the develop branch instead of feature branches.
• Eventually, we want you to do your own commits to the repo instead of pull requests. (I don't like having too many merge commits when they don't make sense.)
• Commit regularly on your machine and squash them.
• Make use of the pre-commit githook I have setup. It runs go fmt on every commit. To do that, run the config command given in the readme.

References:

• https://docs.gitlab.com/ee/topics/gitlab_flow.html
• https://guides.github.com/introduction/flow/index.html
• https://bitcrowd.dev/a-note-on-commit-messages
• https://chris.beams.io/posts/git-commit/
• https://www.atlassian.com/git/tutorials/git-hooks

[npalladium]

I have used the standard log package as this package has been used in handlers.go. However, the logs package has not been used anywhere in the routes directory.

We'll probably get a proper logging package added soon.

[npalladium]

Closed the PR. Sorry. Send a fresh one. Two commits only please. I gave up on @AdaephonBen. Hoping I'll get good git practices from you, @RachitKeertiDas

[RachitKeertiDas]

Roger that.

I have used the standard log package as this package has been used in handlers.go. However, the logs package has not been used anywhere in the routes directory.

We'll probably get a proper logging package added soon.

Will switch to logrus and then re-send a PR