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.

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

Fix/game screen/text/retrieval #47

Closed

Conversation

Shibatch28
Copy link
Contributor

やったこと

  • textの命名規則をtext1.txtのように,text+連番となるように変更
  • ページにアクセスするたびにファイルを取得していたのを廃止

やらないこと

  • 無し

できるようになること(ユーザ目線)

  • 特に変わらない

できなくなること(ユーザ目線)

  • text+Number.txt以外のテキストが課題にならない

動作確認

  • yarn run devにて動くことを確認
  • 他の人のfeat/game-screen/templateブランチへの変更を適用後,ブランチを生やして変更したらyarn run buildが通らなかった

その他

  • Typingの瞬間速度の計算のところでエラー(undefinedになる可能性があるよという内容でした)が起きていたので,直してください.

KinjiKawaguchi and others added 30 commits March 22, 2024 10:35
KinjiKawaguchi and others added 22 commits March 28, 2024 22:05
ci: golangci-lint に on.push のパスを追加した
例えば develop に doc1 から PR 出したときに on pull_request で回すと
doc1 から GitHub Pages にアップロードすることになるけど、それは branch
protection rule に引っかかるのでそれを回避するため
openapiから型定義ファイルを生成するためのテスト
@Shibatch28
Copy link
Contributor Author

何かあると怖いので髙橋さんとかわきん君にも一応レビューリクエストを送りました。お手隙の際に見ていただけると幸いです。

@Shibatch28 Shibatch28 closed this Mar 30, 2024
@Shibatch28 Shibatch28 deleted the fix/game-screen/text/retrieval branch March 30, 2024 10:39
@Shibatch28 Shibatch28 restored the fix/game-screen/text/retrieval branch March 30, 2024 10:39
@Shibatch28 Shibatch28 deleted the fix/game-screen/text/retrieval branch March 30, 2024 10:40
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.

4 participants