Skip to content

Commit

Permalink
[pre-commit.ci] auto fixes from pre-commit.com hooks
Browse files Browse the repository at this point in the history
for more information, see https://pre-commit.ci
  • Loading branch information
pre-commit-ci[bot] authored and gjmooney committed Aug 13, 2024
1 parent 5c11d7e commit a1d053e
Show file tree
Hide file tree
Showing 4 changed files with 31 additions and 23 deletions.
29 changes: 9 additions & 20 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,25 +16,14 @@ jobs:

steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v3

- name: Base Setup
uses: jupyterlab/maintainer-tools/.github/actions/base-setup@v1

- name: Install dependencies
run: python -m pip install -U "jupyterlab>=4.0.0,<5"

- name: Lint the extension
run: |
set -eux
jlpm
jlpm run lint:check
- name: Test the extension
run: |
set -eux
jlpm run test
- name: Build the extension
run: |
set -eux
Expand All @@ -53,7 +42,7 @@ jobs:
pip uninstall -y "jupytercad_gather" jupyterlab
- name: Upload extension packages
uses: actions/upload-artifact@v4
uses: actions/upload-artifact@v3
with:
name: extension-artifacts
path: dist/jupytercad_gather*
Expand All @@ -65,11 +54,11 @@ jobs:

steps:
- name: Install Python
uses: actions/setup-python@v5
uses: actions/setup-python@v4
with:
python-version: '3.9'
architecture: 'x64'
- uses: actions/download-artifact@v4
- uses: actions/download-artifact@v3
with:
name: extension-artifacts
- name: Install and Test
Expand All @@ -96,13 +85,13 @@ jobs:

steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v3

- name: Base Setup
uses: jupyterlab/maintainer-tools/.github/actions/base-setup@v1

- name: Download extension package
uses: actions/download-artifact@v4
uses: actions/download-artifact@v3
with:
name: extension-artifacts

Expand All @@ -119,7 +108,7 @@ jobs:
run: jlpm install

- name: Set up browser cache
uses: actions/cache@v4
uses: actions/cache@v3
with:
path: |
${{ github.workspace }}/pw-browsers
Expand All @@ -136,7 +125,7 @@ jobs:
- name: Upload Playwright Test report
if: always()
uses: actions/upload-artifact@v4
uses: actions/upload-artifact@v3
with:
name: jupytercad_gather-playwright-tests
path: |
Expand All @@ -148,6 +137,6 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 15
steps:
- uses: actions/checkout@v4
- uses: actions/checkout@v3
- uses: jupyterlab/maintainer-tools/.github/actions/base-setup@v1
- uses: jupyterlab/maintainer-tools/.github/actions/check-links@v1
4 changes: 2 additions & 2 deletions .github/workflows/check-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v3
- name: Base Setup
uses: jupyterlab/maintainer-tools/.github/actions/base-setup@v1
- name: Check Release
Expand All @@ -23,7 +23,7 @@ jobs:
token: ${{ secrets.GITHUB_TOKEN }}

- name: Upload Distributions
uses: actions/upload-artifact@v4
uses: actions/upload-artifact@v3
with:
name: jupytercad_gather-releaser-dist-${{ github.run_number }}
path: .jupyter_releaser_checkout/dist
19 changes: 19 additions & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
ci:
# pre-commit.ci will open PRs updating our hooks once a month
autoupdate_schedule: monthly
# skip any check that needs internet access
autofix_prs: true
repos:
# Autoformat and linting, misc. details
- repo: https://github.com/pre-commit/pre-commit-hooks
Expand Down Expand Up @@ -32,3 +37,17 @@ repos:
rev: v4.0.0-alpha.8
hooks:
- id: prettier
args: ['--no-error-on-unmatched-pattern']

- repo: https://github.com/pre-commit/mirrors-eslint
rev: v9.4.0
hooks:
- id: eslint
files: \.tsx?$
types: [file]
additional_dependencies:
- '[email protected]'
- '@typescript-eslint/[email protected]'
- '@typescript-eslint/[email protected]'
- '[email protected]'
- '[email protected]'
2 changes: 1 addition & 1 deletion src/command.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ namespace Private {
return {
Name: newName('GLTF', model),
//@ts-expect-error wip
Object: selected.length > 0 ? selected[0] : objects[0].name ?? '',
Object: selected.length > 0 ? selected[0] : (objects[0].name ?? ''),
Enabled: true
};
},
Expand Down

0 comments on commit a1d053e

Please sign in to comment.