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

Make it easier for developers to find Ballista documentation #330

Merged
merged 1 commit into from
May 14, 2021

Conversation

andygrove
Copy link
Member

Which issue does this PR close?

Closes #329.

Rationale for this change

See issue.

What changes are included in this PR?

  • Added link from top-level README to Ballista README
  • Updated links to user guide to point to user guide source for now, until we have a published version
  • Removed documentation that is no longer relevant (release process)

Are there any user-facing changes?

No

@andygrove andygrove changed the title Make it easer for developers to find Ballista documentation Make it easier for developers to find Ballista documentation May 12, 2021
@andygrove andygrove requested review from alamb and jorgecarleitao May 12, 2021 13:39
@codecov-commenter
Copy link

Codecov Report

Merging #330 (ba3daba) into master (3f269b9) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #330   +/-   ##
=======================================
  Coverage   76.19%   76.19%           
=======================================
  Files         141      141           
  Lines       23786    23786           
=======================================
  Hits        18123    18123           
  Misses       5663     5663           

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 3f269b9...ba3daba. Read the comment docs.

Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Looks good to me -- I clicked around in the Readme's in https://github.com/andygrove/arrow-datafusion/tree/ballista-getting-started and it all looked good to me

@jorgecarleitao jorgecarleitao merged commit 1702d6c into apache:master May 14, 2021
@houqp houqp added ballista documentation Improvements or additions to documentation labels Jul 29, 2021
@andygrove andygrove deleted the ballista-getting-started branch February 6, 2022 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Ballista Getting Started documentation
5 participants