Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
3 changes: 3 additions & 0 deletions .github/workflows/cleanup_closed_pr_packages.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ jobs:
cleanup-package:
name: Cleanup closed PR package
runs-on: ubuntu-latest
permissions:
contents: read # Needed to checkout the repository if the action requires it (though this specific action might not)
packages: write # This is the crucial permission for deleting packages from GHCR
steps:
- name: Cleanup web package
uses: snok/container-retention-policy@v2
Expand Down
14 changes: 8 additions & 6 deletions .github/workflows/manual_test.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
name: On-demand tests
permissions:
contents: read # Required for actions/checkout and reading repo content

on:
workflow_dispatch:
Expand All @@ -10,12 +12,11 @@ env:
jobs:
build_and_test:


strategy:
matrix:
node-version: [18.x, 20.x, 22.x]
node-version: [20.x, 22.x]
operating-system: [ubuntu-latest, windows-latest]
python-version: ['3.10', '3.11', '3.12']
python-version: ['3.11', '3.12']
# See supported Node.js release schedule at https://nodejs.org/en/about/releases/

runs-on: ${{ matrix.operating-system }}
Expand Down Expand Up @@ -43,17 +44,18 @@ jobs:
if: matrix.operating-system == 'ubuntu-latest'
run: npm run start &

- name: Start application in background on Windows
- name: Test application build on Windows
if: matrix.operating-system == 'windows-latest'
run: |
Start-Process -FilePath "npm" -ArgumentList "run start" -NoNewWindow
run: npm run build

- name: Wait for the app to start
if: matrix.operating-system == 'ubuntu-latest'
uses: iFaxity/[email protected]
with:
resource: http://localhost:3000

- name: Run tests
if: matrix.operating-system == 'ubuntu-latest'
uses: nick-fields/retry@v3
with:
max_attempts: 2
Expand Down
5 changes: 3 additions & 2 deletions .github/workflows/node.js.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
name: Node.js CI
permissions:
contents: read

on:
push:
Expand Down Expand Up @@ -62,7 +64,6 @@ jobs:

# when we run this action as depenabot, we need to give it write permissions to the package registry and to the statuses
permissions:
contents: read
packages: write
statuses: write

Expand Down Expand Up @@ -122,7 +123,7 @@ jobs:
- name: Use Node.js
uses: actions/setup-node@v4
with:
node-version: '20.x'
node-version: '22.x'
cache: 'npm'

- name: webfactory/ssh-agent for dev 🖥️
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/packages_retention.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ jobs:
cleanup-packages:
name: Cleanup old GHCR docker packages
runs-on: ubuntu-latest
permissions:
contents: read # Needed to checkout the repository if the action requires it (though this specific action might not)
packages: write # This is the crucial permission for deleting packages from GHCR
steps:
- name: Cleanup web outdated PR packages
uses: snok/container-retention-policy@v2
Expand Down
4 changes: 1 addition & 3 deletions buildPython.js
Original file line number Diff line number Diff line change
Expand Up @@ -90,9 +90,7 @@ const saveFileName = (destFolder, fileName) => {
});

measureTime('Installing build module for python', () => {
executeCommand(
`${venvCommandPrefix} pip install "poetry ~= 1.8.2" ${venvCommandSuffix}`
);
executeCommand(`${venvCommandPrefix} pip install "poetry==1.8.5" ${venvCommandSuffix}`);
Copy link

Copilot AI Apr 24, 2025

Choose a reason for hiding this comment

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

Verify that the strict version pin to poetry 1.8.5 is intentional, as the previous version approximation allowed compatible updates.

Suggested change
executeCommand(`${venvCommandPrefix} pip install "poetry==1.8.5" ${venvCommandSuffix}`);
// Using a version range to allow compatible updates while restricting major version changes.
executeCommand(`${venvCommandPrefix} pip install "poetry>=1.8,<2.0" ${venvCommandSuffix}`);

Copilot uses AI. Check for mistakes.
});

measureTime('Building yaptide_converter', () => {
Expand Down
Loading