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

New max-limit seatbelt #177

Merged
merged 4 commits into from Sep 3, 2017

Conversation

Projects
None yet
3 participants
@ericmjonas
Collaborator

ericmjonas commented Sep 3, 2017

This is to solve #176 for Risecamp @shivaram @ooq could you guys take a quick look

@shivaram

LGTM pending Travis. Minor nit: could we use something other than misc ? Something like scheduler ?

@ooq

This comment has been minimized.

Show comment
Hide comment
@ooq

ooq Sep 3, 2017

Collaborator

LGTM. Should we make it possible to pass the parameter in map()?

Collaborator

ooq commented Sep 3, 2017

LGTM. Should we make it possible to pass the parameter in map()?

@ericmjonas

This comment has been minimized.

Show comment
Hide comment
@ericmjonas

ericmjonas Sep 3, 2017

Collaborator
Collaborator

ericmjonas commented Sep 3, 2017

ericmjonas added some commits Sep 3, 2017

@ericmjonas ericmjonas modified the milestone: v0.3 Sep 3, 2017

@ericmjonas ericmjonas merged commit 6d1670b into master Sep 3, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment