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

Feature/rate limit merge proper #146

Merged
merged 5 commits into from
Jun 23, 2015
Merged

Feature/rate limit merge proper #146

merged 5 commits into from
Jun 23, 2015

Conversation

JuusoV
Copy link
Contributor

@JuusoV JuusoV commented Jun 17, 2015

Added more descriptive commit messages.

},

distributed: {
type: Boolean,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Double check indentation for lines 29-35. While not syntactic, the indentation is important for readability.

@brylie
Copy link
Contributor

brylie commented Jun 17, 2015

Thanks for adding meaningful commit messages :-)

@brylie
Copy link
Contributor

brylie commented Jun 22, 2015

@frenchbread and/or @elnzv, please review.

55 pushed a commit that referenced this pull request Jun 23, 2015
@55 55 merged commit 9ae48a4 into develop Jun 23, 2015
@55 55 removed the in progress label Jun 23, 2015
@55 55 deleted the feature/RateLimitMerge-proper branch June 23, 2015 10:36
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

Successfully merging this pull request may close these issues.

None yet

3 participants