Skip to content

Commit

Permalink
ci: improve deployment environment UX (#2742)
Browse files Browse the repository at this point in the history
* chore: check GitHub action workflows against schema

* chore: simplify environment names

* ci: use same environments

* empty commit

* fix: CI schema
  • Loading branch information
agoose77 authored Oct 5, 2023
1 parent 6c5e182 commit f96b72c
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 7 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/deploy-cpp.yml
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,8 @@ jobs:
permissions:
id-token: write
environment:
name: "deploy-cpp-pypi"
name: "pypi"
url: "https://pypi.org/project/awkward-cpp/"
steps:
- uses: actions/download-artifact@v3
with:
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,8 @@ jobs:
permissions:
id-token: write
environment:
name: "deploy-pypi"
name: "pypi"
url: "https://pypi.org/project/awkward/"
steps:
- uses: actions/download-artifact@v3
with:
Expand Down
7 changes: 3 additions & 4 deletions .github/workflows/docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -193,7 +193,6 @@ jobs:
name: awkward
path: dist/awkward*.whl


build-docs:
runs-on: ubuntu-22.04
needs: [awkward-cpp-wasm, awkward-cpp-x86-64, awkward]
Expand Down Expand Up @@ -311,8 +310,8 @@ jobs:
S3_BUCKET: "preview.awkward-array.org"
DEPLOY_URL: "http://preview.awkward-array.org.s3-website.us-east-1.amazonaws.com"
environment:
name: docs-preview
url: ${{ env.DEPLOY_URL }}/${{ github.head_ref }}
name: docs
url: "${{ env.DEPLOY_URL }}/${{ github.head_ref }}"
steps:
- uses: actions/checkout@v4
- name: Configure AWS credentials
Expand Down Expand Up @@ -343,7 +342,7 @@ jobs:
PRODUCTION_URL: "http://awkward-array.org"
CLOUDFRONT_ID: "EFM4QVENUIXHS"
environment:
name: docs-prod
name: docs
url: ${{ env.PRODUCTION_URL }}${{ steps.sync-main.outputs.path || steps.sync-stable.outputs.path }}
steps:
- uses: actions/checkout@v4
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ jobs:
- '3.9'
- '3.8'

runs-on: macOS-11
runs-on: macos-11

env:
PIP_ONLY_BINARY: cmake,numpy
Expand Down
6 changes: 6 additions & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -77,3 +77,9 @@ repos:
rev: v3.13.0
hooks:
- id: pyupgrade

- repo: https://github.com/python-jsonschema/check-jsonschema
rev: 0.27.0
hooks:
- id: check-github-workflows
args: ["--verbose"]

0 comments on commit f96b72c

Please sign in to comment.