Skip to content

Commit a3f4605

Browse files
committed
Merge branch 'main' into drop_fn_session_param
* main: (185 commits) Resolve path to file when running Shiny Express apps (#1442) Fix changelog formatting test(conftest): Refactor conftest and controls to be imported as shiny.test module (#1413) `make install` shouldn't assume it has `setuptools` (#1437) test(browsers): Unskip webkit tests (#1431) chore: update shiny-vscode extension ID (#1434) Add setuptools as an install requirement in Python 3.12 and above (#1435) Bump version to 0.10.2.9000 v0.10.2 release candidate Use `output_code()` and `render.code()` (#1421) Fix: spinners only show for a split second (#1429) Bump version to 0.10.1.9000 Bump version to 0.10.1 Update CHANGELOG.md (#1419) Update cpu info example bug(CI): Only update the shiny submodule when making the dev docs (#1417) Update shiny CSS (#1415) Bump version to 0.10.0.9000 for development Bump version to 0.10.0 Update shiny JS/CSS assets (#1406) ...
2 parents e5e7525 + bd685bb commit a3f4605

File tree

482 files changed

+26719
-14828
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

482 files changed

+26719
-14828
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
name: 'Custom merge queue browsers'
2+
description: 'Trim down pytest browsers for any github event other than merge_group.'
3+
inputs:
4+
all-browsers:
5+
description: 'Force all pytest browsers to used when testing'
6+
required: false
7+
default: 'false'
8+
outputs:
9+
browsers:
10+
description: 'pytest browsers to use'
11+
value: ${{ steps.browsers.outputs.browsers }}
12+
runs:
13+
using: "composite"
14+
steps:
15+
- name: Determine browsers to use
16+
shell: bash
17+
id: browsers
18+
run: |
19+
if [ "${{ inputs.all-browsers }}" == "true" ]; then
20+
echo "Using all browsers!"
21+
exit 0
22+
fi
23+
24+
if [ "${{ github.event_name }}" == "pull_request" ]; then
25+
echo "Using chrome browser only!"
26+
echo 'browsers=PYTEST_BROWSERS="--browser chromium"' >> "$GITHUB_OUTPUT"
27+
fi
28+
29+
echo "No custom pytest browsers!"
30+
exit 0

.github/py-shiny/setup/action.yaml

+1-2
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ inputs:
44
python-version:
55
description: 'Python version to use'
66
required: false
7-
default: "3.11"
7+
default: "3.12"
88
runs:
99
using: "composite"
1010
steps:
@@ -25,7 +25,6 @@ runs:
2525
shell: bash
2626
run: |
2727
pip install https://github.com/rstudio/py-htmltools/tarball/main
28-
pip install https://github.com/posit-dev/py-shinylive/tarball/main
2928
make install-deps
3029
3130
- name: Install

.github/workflows/build-docs.yaml

+57-12
Original file line numberDiff line numberDiff line change
@@ -1,67 +1,112 @@
1-
name: Build API docs
1+
name: Build API docs and Shinylive for GitHub Pages
22

33
on:
44
workflow_dispatch:
55
push:
66
branches: ["main"]
77
pull_request:
8+
merge_group:
89

910
jobs:
10-
build:
11+
build-docs:
1112
runs-on: ubuntu-latest
1213
strategy:
1314
matrix:
14-
python-version: ["3.11"]
15+
python-version: ["3.12"]
1516
fail-fast: false
1617

1718
steps:
18-
- uses: actions/checkout@v3
19+
- uses: actions/checkout@v4
1920

2021
- name: Set up Python ${{ matrix.python-version }}
21-
uses: actions/setup-python@v4
22+
uses: actions/setup-python@v5
2223
with:
2324
python-version: ${{ matrix.python-version }}
2425

2526
- name: Upgrade pip
2627
run: python -m pip install --upgrade pip
2728

29+
# =====================================================
30+
# API docs
31+
# =====================================================
2832
- name: Install Quarto
2933
uses: quarto-dev/quarto-actions/setup@v2
3034
with:
31-
version: 1.3.340
35+
version: 1.4.549
3236

3337
- name: Install dependencies
3438
run: |
3539
cd docs
3640
make ../venv
37-
. ../venv/bin/activate && pip install https://github.com/posit-dev/py-htmltools/tarball/main https://github.com/posit-dev/py-shinylive/tarball/main
3841
make deps
3942
4043
- name: Run quartodoc
4144
run: |
4245
cd docs
4346
make quartodoc
4447
48+
# =====================================================
49+
# Shinylive
50+
# =====================================================
51+
- name: Check out shinylive
52+
if: github.ref == 'refs/heads/main'
53+
uses: actions/checkout@v4
54+
with:
55+
repository: rstudio/shinylive
56+
ref: main
57+
path: shinylive-repo
58+
59+
- name: Update shinylive's copy of shiny and htmltools
60+
if: github.ref == 'refs/heads/main'
61+
run: |
62+
cd shinylive-repo
63+
make submodules
64+
make submodules-pull-shiny
65+
make submodules-pull-htmltools
66+
67+
- name: Build shinylive
68+
if: github.ref == 'refs/heads/main'
69+
run: |
70+
cd shinylive-repo
71+
make all
72+
73+
# TODO-future; Install updated shinylive assets via updated py-shinylive pkg
74+
75+
# =====================================================
76+
# Build site
77+
# =====================================================
78+
4579
- name: Build site
80+
if: ${{ github.ref == 'refs/heads/main' || github.event_name == 'merge_group' || startsWith(github.head_ref, 'docs') }}
4681
run: |
4782
cd docs
4883
make site
4984
85+
# =====================================================
86+
# Deploy
87+
# =====================================================
88+
89+
- name: Move built API docs and shinylive to single directory
90+
if: github.ref == 'refs/heads/main'
91+
run: |
92+
mkdir deploy
93+
mv docs/_site deploy/docs
94+
mv shinylive-repo/_shinylive deploy/shinylive
95+
5096
- name: Upload site artifact
5197
if: github.ref == 'refs/heads/main'
5298
uses: actions/upload-pages-artifact@v1
5399
with:
54-
path: "docs/_site"
55-
100+
path: "deploy"
56101

57102
deploy:
58103
if: github.ref == 'refs/heads/main'
59-
needs: build
104+
needs: build-docs
60105

61106
# Grant GITHUB_TOKEN the permissions required to make a Pages deployment
62107
permissions:
63-
pages: write # to deploy to Pages
64-
id-token: write # to verify the deployment originates from an appropriate source
108+
pages: write # to deploy to Pages
109+
id-token: write # to verify the deployment originates from an appropriate source
65110

66111
# Deploy to the github-pages environment
67112
environment:

0 commit comments

Comments
 (0)