Skip to content

fix for no python coverage found#33848

Merged
XieYunshen merged 5 commits intoPaddlePaddle:developfrom
XieYunshen:fix_coverage_no_python_coverage_data
Jul 8, 2021
Merged

fix for no python coverage found#33848
XieYunshen merged 5 commits intoPaddlePaddle:developfrom
XieYunshen:fix_coverage_no_python_coverage_data

Conversation

@XieYunshen
Copy link
Contributor

PR types

Others

PR changes

Others

Describe

fix for no python-coverage.data found in coverage ci

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.


set +x
coverage combine `ls python-coverage.data.*`
coverage combine `ls python-coverage.data.*` || echo "no python-coverage.data files were found"
Copy link
Contributor

Choose a reason for hiding this comment

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

发现大家都喜欢用反引号``,而不是用$(...) ,其实用`$(...)`更推荐。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好的

Copy link
Contributor

@lelelelelez lelelelelez left a comment

Choose a reason for hiding this comment

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

LGTM

@XieYunshen XieYunshen merged commit 9f0411f into PaddlePaddle:develop Jul 8, 2021
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.

3 participants