search

saltstack-formulas / node-formula

Manage node.js with SaltStack
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
26 stars 104 forks source link

Markdown link fix #21

Closed Granjow closed 8 years ago

Granjow commented 8 years ago

Sorry, I was about 30 seconds late with my commit, so here another pull request because I messed up the links in the readme. Thanks for merging!

gravyboat commented 8 years ago

No worries, thanks for fixing this! Do you want to rebase so that c66a98a isn't part of this PR?

Granjow commented 8 years ago

Oh, sure. I will try tomorrow. :)

Granjow commented 8 years ago

Okay, I have created a mess before because I did a push --force 30 seconds after you merged, which caused my history to look slightly funny with nearly duplicated commits. Just for general amusement:

treebeforerebasing

git rebase master mdFix then fixed the problem. Thanks for the hint, I would otherwise just have cherry-picked the commit or copy/pasted it on a new branch or used some other workaround. :D Hope it is good now!

gravyboat commented 8 years ago

Looks good to go! Thanks for fixing that up. Rebase is seriously amazing.

Granjow commented 8 years ago

Yes :-] Thanks for the merge!