Skip to content

Revert "chore: migrate to Bun package manager" - #1086

Closed
siiick wants to merge 1 commit into
mainfrom
revert-1063-chore/bun-pm
Closed

Revert "chore: migrate to Bun package manager"#1086
siiick wants to merge 1 commit into
mainfrom
revert-1063-chore/bun-pm

Conversation

@siiick

@siiick siiick commented Sep 5, 2025

Copy link
Copy Markdown
Contributor

Reverts #1063

Comment on lines +13 to +30
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: pnpm/action-setup@v4
- uses: actions/setup-node@v4
with:
cache: pnpm
- run: pnpm install --frozen-lockfile
- run: pnpm --filter tests exec playwright install chromium-headless-shell
- run: pnpm --filter tests run test
- uses: actions/upload-artifact@v4
if: ${{ !cancelled() }}
with:
name: playwright-report
path: |
packages/tests/playwright-report/
packages/tests/trace.zip
retention-days: 1

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}

Copilot Autofix

AI 12 months ago

To fix this, you should add an explicit permissions block to the workflow YAML at the root level (to cover all jobs). The minimal starting point is contents: read, which provides only read access to repository contents. In most test workflows, this suffices unless you are writing to issues, pull requests, packages, etc. In this workflow, none of the steps indicate a need for elevated (write) permissions, so contents: read is appropriate.

Edit the file .github/workflows/playground.yaml: insert a permissions block immediately after the workflow name: and before the on: block. This aligns with YAML conventions and GitHub Actions documentation. No changes to imports or other workflow definitions are required.


Suggested changeset 1
.github/workflows/playground.yaml

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/.github/workflows/playground.yaml b/.github/workflows/playground.yaml
--- a/.github/workflows/playground.yaml
+++ b/.github/workflows/playground.yaml
@@ -1,4 +1,6 @@
 name: E2E tests
+permissions:
+  contents: read
 
 on:
   push:
EOF
@@ -1,4 +1,6 @@
name: E2E tests
permissions:
contents: read

on:
push:
Copilot is powered by AI and may make mistakes. Always verify output.
@siiick siiick closed this Sep 9, 2025
@Pascal-Delange
Pascal-Delange deleted the revert-1063-chore/bun-pm branch February 23, 2026 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants