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

Restore the original sync-web-site.sh #27077

Merged
merged 1 commit into from
Aug 2, 2022

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Aug 2, 2022

Nobody should run this script except me for the releases.

Related to #27076 . The publication of the documentation when releasing was broken.

/cc @ebullient please make sure you haven't documented that people should use this script: they shouldn't by any means. And make sure that the documentation team is aware of this.

  • The main doc is published every night by a separate process
  • This script is used to sync the doc for releases

Nobody should run this script except me for the releases.
@gsmet
Copy link
Member Author

gsmet commented Aug 2, 2022

@ebullient if you need something for the doc team, please create a separate script.

@gsmet gsmet merged commit 1d35f18 into quarkusio:main Aug 2, 2022
@quarkus-bot quarkus-bot bot added this to the 2.12 - main milestone Aug 2, 2022
@ebullient
Copy link
Member

ebullient commented Aug 2, 2022

I did ask about how this worked awhile ago... (my apologies for messing things up, but this is why I was asking). This script (plus old doc instructions) both had comments/references for how to use it to locally build the website, and that is useful to verify some changes.

Can we create a copy of this script in the .github folder for release (where other release-related files and scripts are kept, and are less likely to be bumped), and leave this one for local build?

@gsmet gsmet modified the milestones: 2.12 - main, 2.11.2.Final Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants