assignUser commented on code in PR #14247:
URL: https://github.com/apache/arrow/pull/14247#discussion_r981470126
##########
dev/tasks/docs/github.linux.yml:
##########
@@ -44,23 +44,18 @@ jobs:
ref: {{ default_branch|default("master") }}
path: crossbow
fetch-depth: 1
- - name: Move docs
+ - name: Prepare docs
run: |
# build files are created by the docker user
sudo chown -R ${USER}: build
- PR_DIR=crossbow/docs/pr_docs/{{ pr_number }}
- mkdir -p $PR_DIR
- rsync -a --delete build/docs/ $PR_DIR
- - name: Push changes
+ # These are only used while generating the page
+ rm -rf build/docs/.doctrees
+ - name: Upload preview to S3
+ env:
+ AWS_ACCESS_KEY_ID: {{ '${{ secrets.CROSSBOW_DOCS_AWS_ACCESS_KEY_ID
}}' }}
Review Comment:
No, the github "macro" `${{}}` needs to be wrapped in jinja `{{}}` otherwise
jinja will complain that there is no var named secrets. It's a bit clunky but I
don't know of a better way to prevent jinja from complaining.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
