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: initial attempt to convert docs to use mkdocs #1724

Closed
wants to merge 1 commit into from

Conversation

r3stl355
Copy link
Contributor

Description

I fixed most of the tasks in the issue #1708 but did not make all the changes to docstrings. Submitting this to get early feedback before making larger changes. Please review API Reference -> Delta Table for most changes.

I also suggest changing theme from readthedocs to mkdocs - both built in but latter looks sleeker. Just change the value in mkdocs.yaml to see.

Related Issue(s)

#1708

@github-actions github-actions bot added the binding/python Issues for the Python package label Oct 13, 2023
@github-actions
Copy link

ACTION NEEDED

delta-rs follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message. Please update your PR title and description to match the specification.

@r3stl355 r3stl355 changed the title draft: initial attempt to convert docs to use mkdocs WIP: initial attempt to convert docs to use mkdocs Oct 13, 2023
@r3stl355 r3stl355 changed the title WIP: initial attempt to convert docs to use mkdocs docs: initial attempt to convert docs to use mkdocs Oct 13, 2023
@r3stl355
Copy link
Contributor Author

this is just a draft, I will change everywhere once I get a go ahead but don't want to do all that upfront - that's a lot of potentually unneeded if I missed something

@r3stl355
Copy link
Contributor Author

Please ignore the deleted code when reviewing this - looks like i messed up my merge, focus on docs changes only. I'll make sure all is properly merged before I submit the full PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binding/python Issues for the Python package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant