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.

Sign up for GitHub

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

Jump to bottom

Use curly braces for constructors #7667

Merged
merged 1 commit into from Jul 2, 2019
Merged

Use curly braces for constructors #7667

merged 1 commit into from Jul 2, 2019

Conversation

niboshi
Copy link
Member

@niboshi niboshi commented Jul 2, 2019

For consistency

@niboshi niboshi added the cat:code-fix Code refactoring that does not change the behavior. label Jul 2, 2019
@hvy
Copy link
Member

hvy commented Jul 2, 2019

Jenkins, test this please.

@hvy hvy added the st:test-and-merge State indicating that pull request is approved by a reviewer and can be merged after CI passes. label Jul 2, 2019
@hvy hvy added this to the v7.0.0b2 milestone Jul 2, 2019
@chainer-ci
Copy link
Member

Jenkins CI test (for commit 41e76cb, target branch master) succeeded!

@mergify mergify bot merged commit 8777e90 into chainer:master Jul 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@hvy hvy

Labels
cat:code-fix Code refactoring that does not change the behavior. st:test-and-merge State indicating that pull request is approved by a reviewer and can be merged after CI passes.
Projects
None yet
Milestone
v7.0.0b2
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@niboshi @hvy @chainer-ci