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

Fix CPPLint issues in some tests in fluid/framework #10068

Merged
merged 12 commits into from
Apr 21, 2018
Merged

Fix CPPLint issues in some tests in fluid/framework #10068

merged 12 commits into from
Apr 21, 2018

Conversation

abhinavarora
Copy link
Contributor

No description provided.

@abhinavarora abhinavarora changed the title [DO NOT MERGE]Testing CI Fix CPPLint issues in some tests in fluid/framework Apr 21, 2018
@abhinavarora abhinavarora self-assigned this Apr 21, 2018
kexinzhao
kexinzhao approved these changes Apr 21, 2018
View reviewed changes
Copy link
Contributor

@kexinzhao kexinzhao left a comment

Choose a reason for hiding this comment

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

LGTM!

@abhinavarora abhinavarora merged commit 6402b59 into PaddlePaddle:develop Apr 21, 2018
@abhinavarora abhinavarora deleted the cpplint_framework_contd branch April 21, 2018 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kexinzhao kexinzhao kexinzhao approved these changes

@jacquesqiao jacquesqiao Awaiting requested review from jacquesqiao

@wangkuiyi wangkuiyi Awaiting requested review from wangkuiyi

@sidgoyal78 sidgoyal78 Awaiting requested review from sidgoyal78

@luotao1 luotao1 Awaiting requested review from luotao1

Assignees

@abhinavarora abhinavarora

Labels
Code Cleanup
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@abhinavarora @kexinzhao