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

send upstream PR to rabin-cdc #8

Open
max-mapper opened this issue Jan 9, 2016 · 3 comments
Open

send upstream PR to rabin-cdc #8

max-mapper opened this issue Jan 9, 2016 · 3 comments

Comments

@max-mapper
Copy link
Collaborator

We should contribute the rabin modifications to the upstream source: fd0/rabin-cdc#1

@green-coder
Copy link

How about first merging #4 into this project and then prepare a PR for fd0/rabin-cdc#1 ?

Also, you might want to expose the polynom to the JS API (with get/set modifiers) in case the default doesn't fit some user's use case.

@max-mapper
Copy link
Collaborator Author

@green-coder yes that sounds good, I mentioned that here actually fd0/rabin-cdc#1 (comment)

@GamePad64
Copy link

And then you will be able to make src/ a submodule, synchronized with rabin-cdc upstream

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants