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

docs: Add DataFusion subprojects to navigation menu, other minor updates #10362

Merged
merged 3 commits into from
May 4, 2024

Conversation

andygrove
Copy link
Member

Which issue does this PR close?

N/A

Rationale for this change

Add links to DataFusion subproject to make them more easily discoverable

What changes are included in this PR?

  • Added links to menu
  • Moved bdt from the list of active projects to the list of less active projects
  • Reordered list of projects so that they are in alphabet order

Are these changes tested?

Tested locally

Are there any user-facing changes?

No

:maxdepth: 1
:caption: DataFusion Subprojects

DataFusion Ballista <https://arrow.apache.org/ballista/>
Copy link
Member Author

Choose a reason for hiding this comment

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

Ballista has not yet published its site to the new location

@andygrove andygrove added the documentation Improvements or additions to documentation label May 3, 2024
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. Thank you @andygrove

Note I rendered the docs and the subproject links are so far down I had to scroll down:

Screenshot 2024-05-03 at 7 08 48 AM

Maybe we should reduce the maxdepth for the library guide

@andygrove
Copy link
Member Author

Maybe we could move the ASF Links section to the end?

@alamb
Copy link
Contributor

alamb commented May 3, 2024

Maybe we could move the ASF Links section to the end?

Maybe we could shrink them too (there is no reason for each to get its own vertical line)

Like we could collapse the License / Security into one line maybe

Likewise Donate/Thanks 🤔

Copy link
Contributor

@comphead comphead left a comment

Choose a reason for hiding this comment

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

lgtm thanks @andygrove

@comphead comphead merged commit 2bbfbdf into apache:main May 4, 2024
4 checks passed
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.

4 participants