🎉 Turborepo で再構成 #23
🎉 Turborepo で再構成 #23
Conversation
Walkthroughこの変更は、TypeScript、Next.js、React、内部ライブラリのためのESLint設定の導入、GitHub Actionsワークフローの設定、 Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
![nito-bot[bot]](https://avatars.githubusercontent.com/in/740283?s=60&v=4){kind=small}
There was a problem hiding this comment.
| Device | URL |
|---|---|
| mobile | https://2b377386.nito.pages.dev |
Not what you expected? Are your scores flaky? GitHub runners could be the cause.
Try running on Foo instead
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files ignored due to filter (15)
- apps/frontend/package.json
- apps/frontend/public/favicon.svg
- apps/frontend/public/images/app-store-badge.svg
- apps/frontend/public/images/google-play-badge.png
- apps/frontend/public/images/hero.svg
- apps/frontend/tsconfig.json
- icon.png
- package.json
- packages/eslint-config/package.json
- packages/typescript-config/base.json
- packages/typescript-config/nextjs.json
- packages/typescript-config/package.json
- packages/typescript-config/react-library.json
- tsconfig.json
- turbo.json
Files selected for processing (16)
- .eslintrc.cjs (1 hunks)
- .github/workflows/checks.yml (1 hunks)
- .github/workflows/development.yml (1 hunks)
- .github/workflows/preview.yml (1 hunks)
- .github/workflows/production.yml (1 hunks)
- .gitignore (2 hunks)
- .npmrc (1 hunks)
- .tool-versions (1 hunks)
- README.md (1 hunks)
- apps/frontend/.gitignore (1 hunks)
- apps/frontend/README.md (1 hunks)
- apps/frontend/src/pages/privacy.astro (1 hunks)
- packages/eslint-config/README.md (1 hunks)
- packages/eslint-config/library.js (1 hunks)
- packages/eslint-config/next.js (1 hunks)
- packages/eslint-config/react-internal.js (1 hunks)
Files skipped from review due to trivial changes (5)
- .gitignore
- .tool-versions
- apps/frontend/.gitignore
- apps/frontend/README.md
- packages/eslint-config/README.md
Additional comments: 8
.npmrc (1)
- 1-2: 設定の変更を確認しました。
auto-install-peers=trueはピア依存関係を自動的にインストールするため、必要な依存関係が存在することを保証します。save-exact=trueはpackage.jsonにパッケージの正確なバージョンを保存するため、再現性を高め、マイナーアップデートによる予期しない問題を避けることができます。これらの設定はパッケージ管理のベストプラクティスに沿っているため、変更を承認します。.eslintrc.cjs (1)
- 4-9:
.eslintrc.cjsファイルの設定を確認しました。ignorePatternsによりapps/**とpackages/**ディレクトリが無視されています。これは特定の領域のリントを避ける意図的な設定かもしれませんが、重要なコードのリントを誤ってスキップする可能性もあります。この設定が意図的であり、重要なコードをリントから除外していないことを確認してください。packages/eslint-config/library.js (1)
- 7-8:
library.jsファイルの ESLint 設定を確認しました。pluginsにonly-warnが含まれており、すべてのルール違反をエラーではなく警告として扱います。これは開発中の一時的な措置かもしれませんが、CI/CD パイプラインを通過するコードに潜在的な問題がある可能性を許容するため、本番環境には適していないかもしれません。この設定がプロジェクトのワークフローにとって意図的で適切であることを確認してください。packages/eslint-config/next.js (1)
- 7-20:
next.jsファイルの ESLint 設定を確認しました。extendsにはeslint:recommended,prettier,@vercel/style-guide/eslint/next, およびeslint-config-turboが含まれており、これらは Next.js アプリケーションに推奨される標準的な設定です。env設定にはnode: trueとbrowser: trueの両方が含まれており、サーバーとクライアントの両方でコードを実行する Next.js 環境に適しています。これらの変更は Next.js アプリケーションのベストプラクティスに沿っているため、承認します。packages/eslint-config/react-internal.js (1)
- 17-25:
react-internal.jsファイルの ESLint 設定を確認しました。extendsにはeslint:recommended,prettier, およびeslint-config-turboが含まれており、これらは内部 React ライブラリに推奨される標準的な設定です。env設定にはbrowser: trueが含まれており、ブラウザで実行されることを意図した React ライブラリに適しています。これらの変更は内部 React ライブラリのベストプラクティスに沿っているため、承認します。.github/workflows/checks.yml (1)
- 1-75:
checks.ymlファイルの GitHub Actions ワークフローを確認しました。concurrency設定は、新しいジョブがトリガーされた場合に進行中のジョブをキャンセルするように設定されており、リソースを節約するための良い実践です。changedジョブはpaths-filterアクションを使用してfrontendまたはdocsディレクトリに変更があるかどうかを判断し、条件付きジョブ実行に効率的です。frontendおよびdocsジョブはchangedジョブの出力に基づいて条件付きであり、不要な実行を避けるための良い実践です。これらのジョブには、環境のセットアップ、依存関係のインストール、ビルド、テストの実行といった標準的な CI/CD の実践が含まれています。これらの変更は GitHub Actions ワークフローのベストプラクティスに沿っているため、承認します。README.md (1)
- 1-81:
README.mdファイルのドキュメントを確認しました。新しいプロジェクト構造を反映し、Turborepo 機能の使用方法についての指示が更新されています。アプリとパッケージのビルドおよび開発方法についての明確な指示が提供されており、新しいコントリビューターにとって役立ちます。プロジェクトのための明確で更新された指示を提供しているため、変更を承認します。.github/workflows/production.yml (1)
- 6-132: > Note: This review was outside the patches, so it was mapped to the patch with the greatest overlap. Original lines [1-130]
production.ymlファイルの GitHub Actions ワークフローを確認しました。concurrency設定は、新しいジョブがトリガーされた場合に進行中のジョブをキャンセルするように設定されており、リソースを節約するための良い実践です。changedジョブはpaths-filterアクションを使用してfrontendまたはdocsディレクトリに変更があるかどうかを判断し、条件付きジョブ実行に効率的です。frontendおよびdocsジョブはchangedジョブの出力に基づいて条件付きであり、不要な実行を避けるための良い実践です。これらのジョブには、環境のセットアップ、依存関係のインストール、ビルド、Cloudflare Pages への公開といった標準的な CI/CD の実践が含まれています。これらの変更は本番環境へのデプロイメントのための GitHub Actions ワークフローのベストプラクティスに沿っているため、承認します。
There was a problem hiding this comment.
| Device | URL |
|---|---|
| mobile | https://31876d66.nito.pages.dev |
Not what you expected? Are your scores flaky? GitHub runners could be the cause.
Try running on Foo instead
There was a problem hiding this comment.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (4)
- .github/workflows/checks.yml (1 hunks)
- .github/workflows/development.yml (1 hunks)
- .github/workflows/preview.yml (1 hunks)
- .github/workflows/production.yml (1 hunks)
Files skipped from review as they are similar to previous changes (2)
- .github/workflows/checks.yml
- .github/workflows/production.yml
Additional comments: 6
.github/workflows/development.yml (3)
10-10:
concurrencyグループ名がdevelopment-${{ github.ref }}に変更されました。これにより、同じリファレンスに対するワークフローの同時実行がキャンセルされるようになります。変更は適切に見えます。14-39: 新しい
changedジョブが追加され、特定のステップと出力が設定されています。paths-filterアクションを使用して変更されたパスをフィルタリングし、frontendとdocsのビルドが必要かどうかを判断しています。この変更は Turborepo の構造に合わせて効率的なビルドを実現するためのものです。41-84:
frontendジョブには、新しい条件とステップが追加されています。特に、runs-onがubuntu-22.04に更新され、setup-bunのバージョンがv1.1.1に指定されています。また、bun installとbun run buildのコマンドに--filter=frontendオプションが追加され、cloudflare/pages-actionのバージョンがv1.5.0に更新されています。これらの変更は、新しいビルドシステムに適応するためのものであり、適切に見えます。.github/workflows/preview.yml (3)
9-9:
concurrencyグループ名がpreview-${{ github.ref }}に変更されました。これにより、プルリクエストのプレビュー環境に対するワークフローの同時実行がキャンセルされるようになります。変更は適切に見えます。13-38:
changedジョブが新たに追加され、パスの変更をフィルタリングするステップと、frontendとdocsのビルドが必要かどうかを出力するステップが含まれています。この変更は、効率的なビルドプロセスをサポートするためのものであり、適切に見えます。40-84:
frontendジョブには、新しい条件とステップが追加されています。特に、runs-onがubuntu-22.04に更新され、setup-bunのバージョンがv1.1.1に指定されています。また、bun installとbun run buildのコマンドに--filter=frontendオプションが追加され、cloudflare/pages-actionのバージョンがv1.5.0に更新されています。これらの変更は、新しいビルドシステムに適応するためのものであり、適切に見えます。
Issue
概要
Turborepo で再構成します.
レビュー観点
特になし
レビューレベル
レビュー優先度
参考リンク
スクリーンショット
Summary by CodeRabbit