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.

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

[R][backport-2.2] backport lint fix #21325

Closed
wants to merge 1 commit into from

Conversation

felixcheung
Copy link
Member

What changes were proposed in this pull request?

backport part of the commit that addresses lintr issue

@felixcheung
Copy link
Member Author

also need this in branch-2.1

@felixcheung felixcheung changed the title [R] backport lint fix [R][backport-2.2] backport lint fix May 15, 2018
@SparkQA
Copy link

SparkQA commented May 15, 2018

Test build #90617 has finished for PR 21325 at commit 367d122.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

asfgit pushed a commit that referenced this pull request May 16, 2018
## What changes were proposed in this pull request?

backport part of the commit that addresses lintr issue

Author: Felix Cheung <felixcheung_m@hotmail.com>

Closes #21325 from felixcheung/rlintfix22.
asfgit pushed a commit that referenced this pull request May 16, 2018
## What changes were proposed in this pull request?

backport part of the commit that addresses lintr issue

Author: Felix Cheung <felixcheung_m@hotmail.com>

Closes #21325 from felixcheung/rlintfix22.

(cherry picked from commit 8c223b6)
Signed-off-by: Felix Cheung <felixcheung@apache.org>
MatthewRBruce pushed a commit to Shopify/spark that referenced this pull request Jul 31, 2018
## What changes were proposed in this pull request?

backport part of the commit that addresses lintr issue

Author: Felix Cheung <felixcheung_m@hotmail.com>

Closes apache#21325 from felixcheung/rlintfix22.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants