search

passport-next / passport-mocked

Mock Strategy for Passport.js
MIT License
0 stars 0 forks source link

Change references from passport to @passport-next/passport etc. #1

Closed guyellis closed 6 years ago

guyellis commented 6 years ago

Change references from passport to @passport-next/passport etc.

guyellis commented 6 years ago

@rwky how do you want to manage PRs on these repos? Require at least one approver before merging?

rwky commented 6 years ago

p.s. I just added travis etc, so if you rebase master onto this and push again it should run.

guyellis commented 6 years ago

Thanks for adding Travis. I was going to create an issue for you to do that. You beat me to it @rwky

rwky commented 6 years ago

Can you squash c4070672947ad3a0174b6fdfba193a55c3d6785b into 729e2196c3e32b2a39c4226291d249746f421f8f

And squash 9f658142bd0d0537c3931acafe91e4a287264a11 into 4faee263a4fa8b8eaf69cf9bd63a24eef913df3e into

That'll give us a cleaner commit history.

Travis bailed I've restarted it. Once travis is happy and the commits are squashed this will be good to go.

guyellis commented 6 years ago

@rwky - my apologies. I just squashed it all into one commit without thinking. Is this good to merge now?

rwky commented 6 years ago

Merged and pushed to NPM! @guyellis thanks for the contributions :)