search

jshttp / accepts

Higher-level content negotiation
MIT License
252 stars 42 forks source link

Update Travis CI Node 6.0 test to 6.1 #13

Closed ghost closed 8 years ago

dougwilson commented 8 years ago

Hi @calvinmikael, to make these easier to merge in, can you use the same commit messages as the rest of the Node.js version changes?

ghost commented 8 years ago

@dougwilson Yes, I can make all the commit messages going forward "Add Travis CI tests for Node 5.11 and 6.1" or anything else you feel is descriptive.

dougwilson commented 8 years ago

I would like it to match that of the previous commits. git log .travis.yml to see what they are, please. Once you update this PR with the message, I can merge it :)

ghost commented 8 years ago

@dougwilson Thanks for the clarification. I will update the commit message to be "build: Node.js@6.1". I'd just like to check once beforehand if you're okay with me using a git amend and force push to update this commit. I will also check and change for the other repositories under jshttp that I've submitted PR's for.

dougwilson commented 8 years ago

amend + force push sounds find to me :)

ghost commented 8 years ago

Done

ghost commented 8 years ago

Is this PR still desired or should I close and delete? Sorry.