rohta / bingo_next Goto Github PK

詳細

• 毎週最高１０件PRが作成されるようにする

詳細

-　プロジェクトボードにPRを乗せる際、各項目の設定が必要だが、設定漏れが発生する

• PRテンプレに項目追加し、事前チェックする

チェックの追加項目

• Assignee
• 該当Issueの項目と一致しているか
  • ProjectsのSprint
  • Milestone
• Developmentで該当のIssueを設定したか

サーバーサイド

• Error: Text content does not match server-rendered HTML.
  See more info here: https://nextjs.org/docs/messages/react-hydration-error
• Error: Hydration failed because the initial UI does not match what was rendered on the server.
  See more info here: https://nextjs.org/docs/messages/react-hydration-error
• Error: There was an error while hydrating. Because the error happened outside of a Suspense boundary, the entire root will switch to client rendering.

クライアントサイド

• Warning: Text content did not match. Server: "47" Client: "17"
• Warning: Cannot update a component (Batcher) while rendering a different component (Audios). To locate the bad setState() call inside Audios, follow the stack trace as described in https://reactjs.org/link/setstate-in-render
• Warning: AudioElement: ref is not a prop. Trying to access it will result in undefined being returned. If you need to access the same value within the child component, you should pass it as a different prop. (https://reactjs.org/link/special-props)
• Function components cannot be given refs. Attempts to access this ref will fail. Did you mean to use React.forwardRef()?
• Warning: An error occurred during hydration. The server HTML was replaced with client content in
  .
  See more info here: https://nextjs.org/docs/messages/react-hydration-error
• Uncaught Error: Text content does not match server-rendered HTML.
• Uncaught Error: Hydration failed because the initial UI does not match what was rendered on the server.
• Uncaught Error: There was an error while hydrating. Because the error happened outside of a Suspense boundary, the entire root will switch to client rendering.

再現方法

1. npm run dev
2. classNameを編集する

備考

詳細

• GPL3.0にする
• GitHubが提供するものをそのまま使う

詳細

• dependency-review actionsを使用して、異常時に通知がくるようにする

詳細

• ドキュメントに従いつつ、classNameをひたすら当てていく

詳細

• dom操作と操作に基づくロジックを同一ファイル内に置く
• コンポーネントを跨がないstate管理は、必要であればする
  • またぐstateは本件の対象外とする

子課題

• 配置する
• 単一コンポーネント内でstate管理すべき部分をする
• コンポーネントをまたぐstate管理が必要になった箇所はToDoコメントする

詳細

• npm i
• エラーが出ない状態にする
• 細かいUI設計は取り扱わない

参考リンク

備考

• #57 完了後に行うのが効率的

詳細

• .github/SECURITY.mdを追加する
• 内容は下記
  • dependabot依存であること
  • PRを送ってほしい

備考

• 読みやすさ重視

詳細

• コンポーネントを跨いでstateを管理する必要がある
• reduxは無いので、recoilを導入する

詳細

• #9
• その他のsettinngs項目を設定しきる

備考

• environments項目はGitHub Actions設定時に行う

詳細

• 公式のActionsを用いて、Issue/PRがボードに乗るようにする
• depndabotのPRがボードに乗らない状態を防ぐ

詳細

• 流石に多いので、自動でマージされるようにしたい
• 自動マージならPRの上限件数を増やしてもいいかもしれない

子課題（あれば）

• codeowners書換え
• リポジトリ設定でactionsの許可
• ahmadnassri/action-dependabot-auto-merge@v2を使用してworkflow追加

関連リソース

• https://github.com/ROhta/bingo_next/actions/runs/2636860655

備考

• dependabot.ymlの記述は問題なさそう

詳細

• 移動可能なファイルはsrc配下に移す
• CIやtsconfig等の設定を合わせて修正する

子課題（あれば）

• src配下への移動
• tsconfigのinclude設定
• codeqlの対象ディレクトリ

備考

• 巨大PRになる

詳細

• 公式のactionsを用いて、モジュール関連が更新されるPRの際はdependenciesラベルが付くようにする

詳細

• node18のみで作動するよう、package.jsonに記載する

参考リンク

• 設定

詳細

• GitHubの機能を用いて、リリースノートに以前のリリースからのPRタイトルが列挙されるようにする

子課題（あれば）

• .github/release.ymlを追加
• labelの精査

再現方法

1. dependabotのPRで発火したActionsを見る
2. 発火してない

備考

• GITHUBTOKENのpermissionで、contents　writeが必要っぽい

詳細

• projects betaをいい感じにする
• スクラムボードっぽい感じにする

備考

• 詰まったところ、アップデートに期待するところは、コメント欄に書いていく

詳細

• 全て @ROhta をオーナーとする

詳細

• 数ヶ月動いていないIssueやPRは無かったことにする
• Actionsを用いて自動でクローズされるようにする

詳細

• CodeQLをGitHub Actionsで動作させる

子課題（あれば）

• CodeQLの画面上で確認する

詳細

• Discussionsをいい感じに設定する

備考

• Discussions → Issues → Pull requests の流れをいい感じにしたい

詳細

• dependabotのPRのassigneeをやはりdependabotにしたい
  • 既存PRのassigneeは直さなくていい
• add-to-projectのactionはdependabotのPRではエラーになるので、回避用のif文を付け加える
  • dependabotのPRではactionsがrepository secretsを読み取れない
  • actionsが対応しているtrigerがpull_requestsのみで、pull_request_eventsに対応していない
  • 自前workflowを作ろうにも、ghコマンドがproject betaに対応していないので、CLI操作できない
  • つまり、dependabotPRでadd-to-projectのactionsを実行するとエラー回避手段が無い

• mainブランチをいい感じにする
• require deployments は、GitHub Actions実装時に行う

詳細

• close系3lagelは不要なので、削除する
  • close as not plannedという機能が追加されたので、閉じ方に気をつければ良い
  • 検索時は reason: で絞り込める
• 重複、再現不能、却下を区別する必要があるか、考える
  • 必要な場合はlabelを維持しなければならない

手順

1. Issues → Labelsを開く
2. 該当項目を削除

再現方法

1. dependabotPRのうち、actionsが更新対象になっているものを選ぶ
2. dependencies labelがついていない