Fix up time encoding benchmark #12

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

jgdavey commented Jul 24, 2014

  • Use bmbm for VM warming
  • Update outdated API calls
@jgdavey jgdavey Fix up time encoding benchmark
* Use `bmbm` for VM warming
* Update outdated API calls
64a17ca
Contributor

dchelimsky commented Jul 25, 2014

Hey @jgdavey, thanks for the contribution but, per https://github.com/cognitect/transit-ruby#contributing we're not accepting any pull requests from anybody outside Cognitect in any of the transit repos. Good changes, though, which I'll gladly adopt and credit you for.

dchelimsky closed this Jul 25, 2014

Contributor

dchelimsky commented Jul 25, 2014

BTW - the other 1/2 of that policy is that we do welcome conversation in https://groups.google.com/forum/#!forum/transit-format. So if you have any ideas about design choices (overall or Ruby-specific), implementation details, etc, please feel free to bring them up in that forum.

jgdavey commented Jul 25, 2014

Thanks for the clarification. I should have read the contributing section more closely. I'll redirect any subsequent suggestions to the mailing list.

Thanks!

Contributor

dchelimsky commented Jul 25, 2014

FYI - I moved the https://github.com/cognitect/transit-ruby#contributing higher up the page - will hopefully help save ppl from doing a bunch of work for naught.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment