We probably want to merge as many PRs before doing this to this module, otherwise it is going to make it almost impossible to accept any of the current PRs (especially since this module was not using a format even close to standard before). Perhaps part of the 1 -> 2 version of this module? Thoughts?

As for some of the errors, they should be OK if you want to use the template I linked to in the standard issue (the changes do not really load on my phone well, so not sure).

Oh that's true we should accepted the pull requests before merging this. I'll update it when we start to push towards v2.

Hey @gabeio, if you rebase on top of 16cbfb2 then the __proto__ stuff should be gone now :)

Done 😄 yes the __proto__ error is gone 👍

@dougwilson should I make the changes, just like you made in that commit to this repository also; the package.json & .travis.yml edits that force it to run checks using the eslint?

Yep :)

done 👍 (three of the tests I had to add an if (err) return done(err))

Nice! Doing a rebase should now resolve the second item on the list.

And now item 1 should be resolved as well.

I also just cherry-picked out your commit gabeio@a893cb3 since that is useful outside of just converting to Standard format.

I rebased and squashed them. (without the commit you cherry-picked 👍 )

I'll continue to rebase as the master updates 👍 .

feel free to add the [needs rebase] tag back to this pr as needed.

@dougwilson there are some new errors/suggestions in the standardjs lint:

/home/travis/build/expressjs/session/index.js
  284:15  error  'new Buffer()' was deprecated since v6. Use 'Buffer.alloc()' or 'Buffer.from()' (use 'https://www.npmjs.com/package/safe-buffer' for '<4.5.0') instead  node/no-deprecated-api
  447:27  error  Unexpected mix of '||' and '&&'  no-mixed-operators
  447:65  error  Unexpected mix of '||' and '&&'  no-mixed-operators
/home/travis/build/expressjs/session/test/session.js
  2350:31  error  Unnecessary escape character: \.  no-useless-escape
  2350:37  error  Unnecessary escape character: \.  no-useless-escape

I think I have a stash somewhere with the Buffer change, but not the others.

The unnecessary escape character seems possibly incorrect.

If the . character is within a character class [ ] then it doesn't need to be escaped, which is probably what the issue is.