search

tj / node-migrate

Abstract migration framework for node
MIT License
1.53k stars 221 forks source link

Small typo fix #173

Closed tmikeschu closed 4 years ago

tmikeschu commented 4 years ago

Hello! Noticed this while setting up a new project.

wesleytodd commented 4 years ago

Hey! Thanks for the PR. That was copied from the output of the actual help text, want to fix it there as well?

tmikeschu commented 4 years ago

Good catch, thanks @wesleytodd !

wesleytodd commented 4 years ago

Just an operational note, I have been trying to squash merge when folks have a single PR with multiple commits. My end goal here is to setup conventional commit and auto deploy actions, so it will be important in that case that the merge commit on PRs is clean and follows the format.

Since I have not been able to work on this library for a bit (and have a next major in progress), I have also not documented this. Also, it probably doesn't matter lol. But just wanted to post it in case you cared @LinusU :P

LinusU commented 4 years ago

Nice, yeah that would have been better, will try and remember next time! 👍

Thanks for pointing it out!