-
Notifications
You must be signed in to change notification settings - Fork 7
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
Flutter Webによるdocsビューワの作成 #299
Conversation
Visit the preview URL for this PR (updated for commit 163e289): https://flutter-mobile-project-template-catalog--pr299-docs-we-nhexnejs.web.app (expires Thu, 08 Aug 2024 08:14:44 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23 |
手元の環境起因でlabeler, labelsの生成に失敗していて、CI落ちてますが、その他の部分見ていただけたらと思います |
Ready for review 🚀 |
Cleanupでファイル削除するのを忘れていたので、一度Draftに戻します 🙏 |
Draft解除しました |
Ready for review 🚀 |
@mj-hd
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
コード的には問題なさそうです!
ただ、 CI がこけているのでご確認お願いします🙏
おそらく、ラベルの設定の差分が出ていそうです!
大した修正ではないので、↑自分のほうで対応してコミットしちゃおうと思います!
@@ -3,6 +3,8 @@ removal_list_on_use_template.txt | |||
|
|||
LICENSE.md | |||
|
|||
apps/website |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apps/website/assets/docs
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Webを前提とするプロジェクトなので無効化 | ||
avoid_web_libraries_in_flutter: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a web document viewer. | ||
You can view the markdown documents in the `docs/` directory by this app. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.github/workflows/debug.yml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Conflicts: # .github/labeler.yml # .github/labels.yml
@blendthink すみません、ご対応ありがとうございます! |
概要
レビュー観点
レビューレベル
レビュー優先度
画像 / 動画
確認したこと
動作確認手順
備考
今後のTODO: