Yea, this is from a style guide I'm habitual with so like I said, while poking around I was just kinda doing so out of habit. It's a personal preference, and I agree it should be consistent across a repo.




I'll revert the changes commit and resubmit the PR just removing the unneeded stuff.

—
Sent from Mailbox for iPhone

On Tue, Dec 3, 2013 at 11:45 PM, Alan Shaw <notifications@github.com>
wrote:

>> @@ -33,16 +33,16 @@ module.exports = function (server) {
>>      // POST /report
>>      // Create a vulnerability report
>>      server.route({
>> -        method: 'POST',
>> -        path: '/report',
>> -        handler: function (request) {
>> +        method:     'POST',
> I don't align like this so that I don't have to realign when adding a new var/property that doesn't fit in the space allocated.
> In general I'm not overly concerned with code style. I think it should be consistent within a repo and I tend to follow the style set by the author, as that is what I'd want from contributions to my projects.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/nodesecurity/nodesecurity-api/pull/13/files#r8087699