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

[SPARK-12986][Doc] Fix pydoc warnings in mllib/regression.py #11025

Closed
wants to merge 2 commits into from
Closed

[SPARK-12986][Doc] Fix pydoc warnings in mllib/regression.py #11025

wants to merge 2 commits into from

Conversation

nampham2
Copy link
Contributor

@nampham2 nampham2 commented Feb 2, 2016

I have fixed the warnings by running "make html" under "python/docs/". They are caused by not having blank lines around indented paragraphs.

@mengxr
Copy link
Contributor

mengxr commented Feb 3, 2016

ok to test

@mengxr
Copy link
Contributor

mengxr commented Feb 3, 2016

LGTM pending Jenkins.

@SparkQA
Copy link

SparkQA commented Feb 3, 2016

Test build #50626 has finished for PR 11025 at commit 5708100.

  • This patch fails Python style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@mengxr
Copy link
Contributor

mengxr commented Feb 3, 2016

@nampham2 You can run dev/lint-python to check python code style locally. Could you fix the errors and update this PR? Thanks!

@nampham2
Copy link
Contributor Author

nampham2 commented Feb 3, 2016

Thanks @mengxr! I did not know about this lint-python :). The PR is updated.

@SparkQA
Copy link

SparkQA commented Feb 3, 2016

Test build #50676 has finished for PR 11025 at commit 812470d.

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

@holdenk
Copy link
Contributor

holdenk commented Feb 8, 2016

looks good to me as well and seems to pass pylint now @mengxr

@mengxr
Copy link
Contributor

mengxr commented Feb 8, 2016

Merged into master. Thanks!

@asfgit asfgit closed this in edf4a0e Feb 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants