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

Add documentation about non-Koalas APIs #1420

Merged
merged 5 commits into from
Apr 15, 2020

Conversation

beobest2
Copy link
Contributor

@beobest2 beobest2 commented Apr 14, 2020

Resolves #1414
I have added a preemptive guide to non-Koalas APIs to best practeces.
Please check.

@codecov-io
Copy link

codecov-io commented Apr 14, 2020

Codecov Report

Merging #1420 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1420   +/-   ##
=======================================
  Coverage   95.13%   95.13%           
=======================================
  Files          34       34           
  Lines        7958     7958           
=======================================
  Hits         7571     7571           
  Misses        387      387           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d63e747...ac64902. Read the comment docs.

Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

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

I added some suggestions on the comments. Feel free to rephrase or reword

Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

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

Looks okay.

Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

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

Copy link
Collaborator

@ueshin ueshin left a comment

Choose a reason for hiding this comment

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

Otherwise, LGTM.

@HyukjinKwon
Copy link
Member

Merged. Thanks!

@HyukjinKwon HyukjinKwon merged commit cd500d2 into databricks:master Apr 15, 2020
@beobest2 beobest2 deleted the add_doc branch May 13, 2020 12:14
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.

Document that we don't support the compatibility with non-Koalas APIs yet.
4 participants