Skip to content

2025 Updates #47

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.

Sign up for GitHub

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

Open
Sparticuz wants to merge 9 commits into
base: master
Choose a base branch
from
Open

2025 Updates #47

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
bd21b36
Dump old codeql
Sparticuz Dec 5, 2025
1c29b05
Update deps
Sparticuz Dec 5, 2025
2b02c42
Update infra
Sparticuz Dec 5, 2025
4e744cd
Fix lint errors
Sparticuz Dec 5, 2025
38a7562
Continue so the code path doesn't continue
Sparticuz Dec 5, 2025
d1062e3
Update qpdf
Sparticuz Dec 5, 2025
386e9f4
min node 18
Sparticuz Dec 5, 2025
77d2789
min node 20
Sparticuz Dec 5, 2025
16a163a
js interop
Sparticuz Dec 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
41 changes: 0 additions & 41 deletions .github/workflows/codeql-analysis.yml
View file
Open in desktop

This file was deleted.

8 changes: 4 additions & 4 deletions .github/workflows/node.js.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,17 +15,17 @@ jobs:

strategy:
matrix:
node-version: [16.x, 18.x, 20.x, 22.x, latest]
node-version: [20.x, 22.x, 24.x, latest]

steps:
- name: Install QPDF
run: |
sudo apt-get install wget unzip -y
wget -O /tmp/qpdf.zip https://github.com/qpdf/qpdf/releases/download/v11.9.1/qpdf-11.9.1-bin-linux-x86_64.zip
wget -O /tmp/qpdf.zip https://github.com/qpdf/qpdf/releases/download/v12.2.0/qpdf-12.2.0-bin-linux-x86_64.zip
sudo unzip -d / /tmp/qpdf.zip
- uses: actions/checkout@v4
- uses: actions/checkout@v6
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: ${{ matrix.node-version }}
- run: npm ci
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/npm-publish.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,8 @@ jobs:
publish-npm:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
- uses: actions/checkout@v6
- uses: actions/setup-node@v6
- run: npm ci
- run: npm run build
- run: npm publish
Expand Down
4 changes: 2 additions & 2 deletions eslint.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
// @ts-check

import myConfig from "@sparticuz/eslint-config";
import tseslint from "typescript-eslint";
import { defineConfig } from "eslint/config";
Copy link

Copilot AI Dec 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The import import { defineConfig } from "eslint/config" may not be valid. As of January 2025, ESLint doesn't export from this path. The typical approach for ESLint flat config is to either:

  1. Use plain objects without a helper function, or
  2. Use TypeScript with JSDoc type annotations for type safety

Please verify that this import path exists in ESLint 9.39.1. If not, consider removing the defineConfig wrapper or using a valid import.

Copilot uses AI. Check for mistakes.

export default tseslint.config(
export default defineConfig(
{
ignores: ["dist", "coverage"],
},
Expand Down
Loading