Skip to content

Convert docs source to markdown #18375

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

Merged
merged 1 commit into from
Jul 27, 2023
Merged

Conversation

mosabua
Copy link
Member

@mosabua mosabua commented Jul 21, 2023

Description

Continued conversion of docs source to markdown/myst

  • All files in language folder
  • All files in sql folder
  • Adjust test for reserved keyword list

Verified that the output HTML is identical or only differs in whitespace, wrapping and other aspects that do not impact the rendered, visible output.

Additional context and related issues

#17732
#17742
#17759

Release notes

(x) This is not user-visible or docs only and no release notes are required.

@cla-bot cla-bot bot added the cla-signed label Jul 21, 2023
@github-actions github-actions bot added the docs label Jul 21, 2023
@mosabua mosabua changed the title Convert docs source to markdown [WIP] Convert docs source to markdown Jul 21, 2023
@mosabua mosabua force-pushed the mm/more-md branch 2 times, most recently from 177adb7 to b0a9262 Compare July 24, 2023 20:56
@mosabua mosabua changed the title [WIP] Convert docs source to markdown Convert docs source to markdown Jul 24, 2023
@mosabua mosabua requested a review from electrum July 24, 2023 20:59
Copy link
Member

@ebyhr ebyhr left a comment

Choose a reason for hiding this comment

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

Some pages are broken. e.g.
Screenshot 2023-07-25 at 12 16 45

@mosabua
Copy link
Member Author

mosabua commented Jul 25, 2023

Some pages are broken. e.g. Screenshot 2023-07-25 at 12 16 45

Whoa .. thats weird.. worked in my testing. Let me check some more.

@mosabua
Copy link
Member Author

mosabua commented Jul 25, 2023

Fixed the issues .. the comment for START inside the table for some reason did not work in new testing .. the sphinx iterative builds are a pain sometimes.. I also adjusted the test .. all should work now

@mosabua
Copy link
Member Author

mosabua commented Jul 25, 2023

Build failure is an unrelated Iceberg test.

@mosabua
Copy link
Member Author

mosabua commented Jul 26, 2023

Also adjusted for merge of #18431

- All files in language folder
- All files in sql folder
- Adjust test for reserved keyword list
@mosabua mosabua merged commit 883cc17 into trinodb:master Jul 27, 2023
@mosabua mosabua deleted the mm/more-md branch July 27, 2023 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants