-
-
Notifications
You must be signed in to change notification settings - Fork 175
Bring over all the Trainings configuration and tools #1141
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
Conversation
- Begin replacing "Trainings" with "Documentation"
✔️ Deploy Preview for 6-dev-docs-plone-org ready! 🔨 Explore the source changes: c322350 🔍 Inspect the deploy log: https://app.netlify.com/sites/6-dev-docs-plone-org/deploys/61ebd2d7f28d4f00083f700f 😎 Browse the preview: https://deploy-preview-1141--6-dev-docs-plone-org.netlify.app |
This is a WIP. I still have a bit more work to do before it is ready for others to start using. But if you just can't wait, check out this branch and read |
- Update search select menu options - Add another Plone logo version - Blackify
Remove theme-package because it was added out of confusion
Update index for structure Fix Sphinx warnings and errors
So you can skip the line |
It's wrong what I wrote, I did a test from scratch. But I think it's fine now. We can come back to it later. |
@ksuess, I added one more commit for you to test out. You'll need to run If it is OK to merge this PR as is, please let me know. I am eager to get something deployed for people to start playing with. |
This can be deleted: .PHONY: test |
Move `all` target to the end
Good catch! Fixed in 6941292. |
|
Maybe just add a text hint with .PHONY: build |
Let's see what other reviewers find. From me a GO! |
I you add
then |
Add description comment for livehtml when calling `make`
I added updates per your comments, but grabbed the description from https://pypi.org/project/sphinx-autobuild/ for |
As long as this PR is open: here another little one: |
I updated the robots.txt, based on what shows up in |
I'm going to merge this branch after Europeans have had one more day to review it. I want to announce that |
# id: deploy | ||
# uses: Pendect/action-rsyncer@v1.1.0 | ||
# env: | ||
# DEPLOY_KEY: ${{secrets.DEPLOY_KEY_TRAINING}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This probably needs a new key
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, that's why I commented it out. Who can provide one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ericof should be able to do so.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's continue the conversion in #1146
# options: '' | ||
# ssh_options: '-p ${{secrets.DEPLOY_PORT}}' | ||
# src: '_build/html/' | ||
# dest: '${{secrets.DEPLOY_USER_TRAINING}}@${{secrets.DEPLOY_SERVER_TRAINING}}:${{secrets.DEPLOY_PATH_TRAINING}}/5' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here also
…tation into port-training-config
See checklist in #1134