Skip to content
This repository has been archived by the owner on Sep 14, 2022. It is now read-only.

Update split token generation/validation branch from master #91

Open
wants to merge 4 commits into
base: split-module
Choose a base branch
from

Conversation

strawbrary
Copy link

Merged the latest changes from the master branch, resolved the conflicts, fixed some styles that were inconsistent with the existing code and added some simple documentation for the new API to the README.

@strawbrary strawbrary changed the title Update from master, add new API to README Update split token generation/validation branch from master Mar 25, 2016
@dougwilson dougwilson added the pr label May 23, 2016
@dougwilson dougwilson self-assigned this May 23, 2016
@twelve17
Copy link

twelve17 commented Nov 8, 2016

Any updates on this?

@strawbrary
Copy link
Author

@twelve17 My guess is @dougwilson still has a long backlog of PRs to review given how many projects he manages. If you want to try it out, I've published the fork temporarily on npm under the name csurfer.

@dougwilson
Copy link
Contributor

I do, and I got an opportunity to start plowing through them as of a week ago, so will get getting here after not too long. Really, the split-module branch just needs to be rebased, as merges are not part of this org's workflow anyway :)

@strawbrary
Copy link
Author

Oh ok I wasn't aware of that. I'll try to submit a new PR soon following the correct protocol.

@dougwilson
Copy link
Contributor

It's no problem. I'm not sure if you actually can do that work via PR at all.

@dougwilson dougwilson force-pushed the split-module branch 3 times, most recently from cb74e9b to f1a3127 Compare November 20, 2016 02:17
@dougwilson dougwilson added this to the 2.0.0 milestone Apr 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants