Skip to content
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

Jump to bottom

yarn to bun #64

Merged
merged 4 commits into from
Feb 4, 2024
Merged

yarn to bun #64

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
bf9058c
chore: use bun
tktcorporation Feb 4, 2024
8a8c332
chore: use antfu/ni
tktcorporation Feb 4, 2024
75f800a
chore: fix https://github.com/vitejs/vite/issues/15714
tktcorporation Feb 4, 2024
8e8c775
chore: trust simple-git-hooks
tktcorporation Feb 4, 2024
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
12 changes: 6 additions & 6 deletions .github/wip-screenshot.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,16 +14,16 @@ jobs:
- name: Checkout code
uses: actions/checkout@v3

- name: Setup Node.js
uses: actions/setup-node@v3
with:
node-version-file: 'package.json'
- name: setup bun
uses: oven-sh/setup-bun@v1

- run: npm i -g @antfu/ni
Copy link
Contributor

Choose a reason for hiding this comment

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

npm i -g @antfu/ni の実行は、Bun を使用しているプロジェクトで npm コマンドを直接使用することの一貫性の欠如を示しています。bun install @antfu/ni または ni の Bun によるサポートを確認してください。


- name: Install dependencies
run: yarn install --frozen-lockfile
run: nci
Copy link
Contributor

Choose a reason for hiding this comment

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

依存関係のインストールに nci を使用することは、antfu/ni を介して異なるパッケージマネージャー間でのコマンドの抽象化を意図していますが、Bun で直接 bun install を使用することが推奨されます。nci の使用は冗長かもしれません。


- name: Start web app (example using Vite)
run: yarn dev &
run: nr dev &
Copy link
Contributor

Choose a reason for hiding this comment

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

Web アプリを起動するコマンドが nr dev & に更新されました。nr コマンドの使用は antfu/ni に依存していますが、このタスクに Bun のネイティブコマンドを使用することも検討してください。また、バックグラウンドで実行する場合は、適切なリソース管理を確認してください。


- name: Run Puppeteer script to capture screenshot
id: capture-screenshot
Expand Down
16 changes: 8 additions & 8 deletions .github/workflows/lint-test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,16 +11,16 @@ jobs:
- name: Checkout code
uses: actions/checkout@v4

- name: setup node
uses: actions/setup-node@v3
with:
node-version-file: 'package.json'
- name: setup bun
uses: oven-sh/setup-bun@v1

- run: npm i -g @antfu/ni
Copy link
Contributor

Choose a reason for hiding this comment

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

npm i -g @antfu/ni の実行は、Bun を使用しているプロジェクトで npm コマンドを直接使用するのは一貫性がなく、混乱を招く可能性があります。bun install @antfu/ni または ni 自体の Bun によるサポートを確認してください。


- name: install dependencies
run: yarn install --frozen-lockfile
run: nci
Copy link
Contributor

Choose a reason for hiding this comment

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

依存関係のインストールに nci を使用することは、antfu/ni を介して異なるパッケージマネージャー間でのコマンドの抽象化を意図していますが、Bun で直接 bun install を使用することが推奨されます。nci の使用は冗長かもしれません。


- name: lint
run: yarn lint && yarn type-check
run: nr lint && nr type-check
Copy link
Contributor

Choose a reason for hiding this comment

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

Lint と型チェックのコマンドが nr lint && nr type-check に更新されました。nr コマンドの使用は antfu/ni に依存していますが、これらのタスクに Bun のネイティブコマンドを使用することも検討してください。また、コマンドの実行に失敗した場合にビルドが停止するように、&& の代わりに ; を使用することを検討してください。


- name: test
run: yarn test
run: nr test
Copy link
Contributor

Choose a reason for hiding this comment

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

テストコマンドが nr test に更新されました。ここでも、Bun のネイティブコマンドの使用を検討するか、antfu/ni を介したコマンドの抽象化が適切かどうかを再評価してください。

12 changes: 6 additions & 6 deletions .github/workflows/tag-on-push.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,16 +15,16 @@ jobs:
with:
fetch-depth: 0

- name: setup node
uses: actions/setup-node@v3
with:
node-version-file: 'package.json'
- name: setup bun
uses: oven-sh/setup-bun@v1

- run: npm i -g @antfu/ni
Copy link
Contributor

Choose a reason for hiding this comment

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

npm i -g @antfu/ni の実行は、Bun を使用しているプロジェクトで npm コマンドを直接使用することの一貫性の欠如を示しています。bun install @antfu/ni または ni の Bun によるサポートを確認してください。


- name: install dependencies
run: yarn install --frozen-lockfile
run: nci
Copy link
Contributor

Choose a reason for hiding this comment

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

依存関係のインストールに nci を使用することは、antfu/ni を介して異なるパッケージマネージャー間でのコマンドの抽象化を意図していますが、Bun で直接 bun install を使用することが推奨されます。nci の使用は冗長かもしれません。


- name: lint
run: yarn lint && yarn type-check
run: nr lint && nr type-check
Copy link
Contributor

Choose a reason for hiding this comment

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

Lint と型チェックのコマンドが nr lint && nr type-check に更新されました。nr コマンドの使用は antfu/ni に依存していますが、これらのタスクに Bun のネイティブコマンドを使用することも検討してください。また、コマンドの実行に失敗した場合にビルドが停止するように、&& の代わりに ; を使用することを検討してください。


create_tag:
runs-on: ubuntu-latest
Expand Down
12 changes: 6 additions & 6 deletions .github/workflows/upload-build-files.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,16 +22,16 @@ jobs:
with:
fetch-depth: 0

- name: Setup Node.js
uses: actions/setup-node@v3
with:
node-version-file: 'package.json'
- name: setup bun
uses: oven-sh/setup-bun@v1

- run: npm i -g @antfu/ni
Copy link
Contributor

Choose a reason for hiding this comment

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

npm i -g @antfu/ni の実行は、Bun を使用しているプロジェクトで npm コマンドを直接使用することの一貫性の欠如を示しています。bun install @antfu/ni または ni の Bun によるサポートを確認してください。


- name: Install dependencies
run: yarn install --frozen-lockfile
run: nci
Copy link
Contributor

Choose a reason for hiding this comment

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

依存関係のインストールに nci を使用することは、antfu/ni を介して異なるパッケージマネージャー間でのコマンドの抽象化を意図していますが、Bun で直接 bun install を使用することが推奨されます。nci の使用は冗長かもしれません。


- name: Build Electron app
run: yarn dist
run: nr dist
Copy link
Contributor

Choose a reason for hiding this comment

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

Electron アプリのビルドコマンドが nr dist に更新されました。ここでも、Bun のネイティブコマンドの使用を検討するか、antfu/ni を介したコマンドの抽象化が適切かどうかを再評価してください。


- name: Upload artifacts
uses: actions/upload-artifact@v3
Expand Down
Binary file removed Screenshot.jpg
View file
Open in desktop
Binary file not shown.
Binary file added bun.lockb
View file
Open in desktop
Binary file not shown.
5 changes: 3 additions & 2 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
},
"productName": "add-metadata-to-vrc-photo",
"scripts": {
"preinstall": "npx only-allow yarn",
"preinstall": "npx only-allow bun",
"postinstall": "simple-git-hooks",
"setup:debug": "bash debug/genDevFiles.sh",
"dev": "eval `dbus-launch --sh-syntax` && concurrently \"nr dev:vite\" \" nr dev:electron\"",
Expand All @@ -41,8 +41,9 @@
},
"engines": {
"node": "20",
"yarn": "1"
"bun": "1"
},
"trustedDependencies": ["simple-git-hooks"],
"dependencies": {
"@heroicons/react": "^2.0.18",
"@radix-ui/react-alert-dialog": "^1.0.5",
Expand Down
2 changes: 1 addition & 1 deletion tsconfig.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"lib": ["DOM", "DOM.Iterable", "ESNext"],
"types": ["vite/client", "node", "electron", "jest"],
"allowJs": false,
"skipLibCheck": false,
"skipLibCheck": true,
"esModuleInterop": false,
"allowSyntheticDefaultImports": true,
"strict": true,
Expand Down
Loading