Skip to content

Hide globals & redesign tls#23066

Closed
pawelpiotrowicz wants to merge 0 commit intoPaddlePaddle:developfrom
pawelpiotrowicz:develop
Closed

Hide globals & redesign tls#23066
pawelpiotrowicz wants to merge 0 commit intoPaddlePaddle:developfrom
pawelpiotrowicz:develop

Conversation

@pawelpiotrowicz
Copy link
Contributor

@jczaja
Copy link
Contributor

jczaja commented Mar 17, 2020

@pawelpiotrowicz To have tests started , you need to put in your commit message "test=develop" string next to other info

jczaja
jczaja previously approved these changes Mar 17, 2020
Copy link
Contributor

@jczaja jczaja left a comment

Choose a reason for hiding this comment

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

LGTM

@lidanqing-vv
Copy link
Contributor

@pawelpiotrowicz Maybe you need to install pre-commit because integration CI failed. https://pre-commit.com/

@luotao1
Copy link
Contributor

luotao1 commented Mar 24, 2020

Paddle _ PR_CI_Coverage _ #53760 (19 Mar 20 10_22) _ Diff Coverage — TeamCity.pdf

@lidanqing-vv
Copy link
Contributor

@luotao1 Hi, will you add a UT for this PR

@luotao1
Copy link
Contributor

luotao1 commented Mar 31, 2020

Yes, I will add a UT, but I need some time.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants