Skip to content
This repository was archived by the owner on Nov 1, 2024. It is now read-only.

Conversation

@andrewaikens87
Copy link
Contributor

Summary:
Improves test coverage for icolumn.py through test_string_column.

icolumn.py has ~55% code coverage currently and this diff brings it to 78.3.

Differential Revision: D37289878

andrewaikens87 and others added 3 commits June 29, 2022 10:42
Differential Revision: D37492649

fbshipit-source-id: 2d71c5002a002fa478359bb551e66a97a7365e52
Differential Revision: D37492647

fbshipit-source-id: a875d5fdd7499566541e2623a9581f7ed1812974
Summary:
Improves test coverage for icolumn.py through test_string_column.

icolumn.py has ~55% code coverage currently and this diff brings it to 78.3.

Differential Revision: D37289878

fbshipit-source-id: 11958d7708c90878a6cfa6ca2a812774cc1dd6c5
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Jun 29, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D37289878

@andrewaikens87 andrewaikens87 deleted the export-D37289878 branch June 29, 2022 20:54
@andrewaikens87 andrewaikens87 restored the export-D37289878 branch June 29, 2022 20:58
@andrewaikens87 andrewaikens87 deleted the export-D37289878 branch June 29, 2022 20:58
andrewaikens87 added a commit to andrewaikens87/torcharrow that referenced this pull request Jun 29, 2022
Summary:
Pull Request resolved: pytorch#410

Improves test coverage for icolumn.py through test_string_column.

icolumn.py has ~55% code coverage currently and this diff brings it to 78.3.

Differential Revision: D37289878

fbshipit-source-id: ad6e74c5d23b0ca30c39168740533cc169dd0bd2
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants