Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include other patches and then release? #2

Open
mvhconsult opened this issue Aug 11, 2016 · 2 comments
Open

Include other patches and then release? #2

mvhconsult opened this issue Aug 11, 2016 · 2 comments

Comments

@mvhconsult
Copy link

I actively use this library, would be nice to formalize it a bit.

Examples of people adding the code to Github:
https://github.com/infomaniac50/SimpleTimer (with some patches, and forked a lot)
https://github.com/hamityanik/SimpleTimer (Fork of the above with most patches it seems)
https://github.com/schinken/SimpleTimer (with additional option for expressions)
https://github.com/kadu/SimpleTimer (?)
https://github.com/MeULEDs/SimpleTimer (?)
https://github.com/Timendainum/SimpleTimer (other version?)

@schinken
Copy link
Owner

What do you suggest? Merge the patchs from infomaniac50 and hamityanik, or merge my std::function patch to infomaniac50?

The important part is: Adding the active fork to the library manager

I once contacted the original author of this library, without success

@mvhconsult
Copy link
Author

Yes, I am basically suggesting to either nag infomaniac50 into serving as the main maintainer, or do it here (if you don't mind). I can put work into merging some stuff, as long as we are two doing the testing.

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

No branches or pull requests

2 participants