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

Updates for Docusaurus 2 #406

Merged
merged 6 commits into from
Jul 26, 2023
Merged

Updates for Docusaurus 2 #406

merged 6 commits into from
Jul 26, 2023

Conversation

vtlim
Copy link
Member

@vtlim vtlim commented Jul 26, 2023

Changes in this PR

  • Removed _data (content now in static/js)
  • Updated Python publishing script (see details below)
  • Updated link in druid-powered.md since the file moved
  • Updated CSS for styling improvements

Script update details

Makes the source directory a command line argument, shows the full path of the source directory during verification step, and some cleanup.

For an invalid source, the output looks like this

$ python do_all_things.py -v 26.0.0

Checking docs source from the following:
Repo:	'/Users/victorialim/Documents/work_repos/druid'
Error: Supply a valid path for apache/druid in the '--source' flag

For a valid source, the output looks like this

$ python do_all_things.py -v 26.0.0 --source ../../druid-apache 

Checking docs source from the following:
Repo:	'/Users/victorialim/Documents/work_repos/druid-apache'
Branch:	'docusaurus2'

Is the listed docs source correct? (y/n) y
Is 26.0.0 going to be the highest version available for download? If yes, the docs will also be used for 'latest'. (y/n) y
Also copying the docs to docs/latest.
Installing Docusaurus 2...

@vtlim vtlim changed the title Update Docusaurus 2 publishing script and some CSS Updates for Docusaurus 2 Jul 26, 2023
@vogievetsky
Copy link
Contributor

Thank you for the changes!

@vogievetsky vogievetsky merged commit 510b39a into apache:master Jul 26, 2023
@vtlim vtlim deleted the script-update branch August 1, 2023 22:27
@317brian
Copy link
Contributor

317brian commented Aug 4, 2023

@vtlim we forgot to update the README for this change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants