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

Feature/disenable copilot on use template #110

Merged
merged 9 commits into from
Feb 2, 2024

Conversation

Yamasaki-pan961
Copy link
Contributor

@Yamasaki-pan961 Yamasaki-pan961 commented Feb 1, 2024

概要

レビュー観点

Actionsの改善点があれば指摘いただけると助かります。

レビューレベル

  • Lv1: ぱっとみて違和感がないかチェックして Approve する
  • Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
  • Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

  • すぐに見てもらいたい ( hotfix など ) 🚀
  • 今日中に見てもらいたい 🚗
  • 今日〜明日中で見てもらいたい 🚶
  • 数日以内で見てもらいたい 🐢

画像 / 動画

スキップ

Before After Design

動作確認手順

このプロジェクトをForkしたリポジトリからテンプレート機能を使ってリポジトリを作成したので、
その際のワークフローをご確認ください。

備考

@Yamasaki-pan961 Yamasaki-pan961 marked this pull request as ready for review February 1, 2024 09:27
Copy link

github-actions bot commented Feb 1, 2024

Ready for review 🚀

Copy link
Member

@blendthink blendthink left a comment

Choose a reason for hiding this comment

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

@Yamasaki-pan961
少しだけコメントさせていただきました、ご確認お願いします🙏

Comment on lines +16 to +17
// GitHubのテンプレート機能使用してリポジトリを作った際に、
// copilotの設定は無効化されるように上書きするワークフローが実行されます
Copy link
Member

Choose a reason for hiding this comment

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

good-badge
このようなコメント助かります🥰

.github/actions/cleanup-copilot-settings/action.yaml Outdated Show resolved Hide resolved
Comment on lines 23 to 26
"github.copilot.editor.enableAutoCompletions": true,
"github.copilot.editor.enableAutoCompletions": true
Copy link
Member

Choose a reason for hiding this comment

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

ask-badge
このフォーマットをかけるのは必要なのでしょうか?

もし不要であれば無理して修正しなくてもいいかもしれないと思いました、、!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

フォーマットは不要です。

ただ自分のJSONフォーマッタだけおかしくて、、、

VSCodeのJSON言語標準フォーマッターを自分と @morikann で比べてみると、
自分だけ最終行の改行がきえちゃうんですよね、、

なのでPrettierを導入してフォーマットしたのでこのようになりました。

Copy link
Member

Choose a reason for hiding this comment

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

この辺り、明日か来週のどこかでヒアリングさせてください、、!

Copy link
Contributor Author

@Yamasaki-pan961 Yamasaki-pan961 left a comment

Choose a reason for hiding this comment

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

@blendthink
コメントありがとうございます!
修正いたしました!

Comment on lines 23 to 26
"github.copilot.editor.enableAutoCompletions": true,
"github.copilot.editor.enableAutoCompletions": true
Copy link
Contributor Author

Choose a reason for hiding this comment

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

フォーマットは不要です。

ただ自分のJSONフォーマッタだけおかしくて、、、

VSCodeのJSON言語標準フォーマッターを自分と @morikann で比べてみると、
自分だけ最終行の改行がきえちゃうんですよね、、

なのでPrettierを導入してフォーマットしたのでこのようになりました。

.github/actions/cleanup-copilot-settings/action.yaml Outdated Show resolved Hide resolved
Copy link
Member

@blendthink blendthink left a comment

Choose a reason for hiding this comment

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

コメントしているものの、別で確認させていただく & たいしたことではないため Approve させていただきます!
LGTM!

Copy link
Member

@blendthink blendthink left a comment

Choose a reason for hiding this comment

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

LGTM

@Yamasaki-pan961 Yamasaki-pan961 merged commit 4d36ce8 into main Feb 2, 2024
3 checks passed
@Yamasaki-pan961 Yamasaki-pan961 deleted the feature/disenable-copilot-on-use-template branch February 2, 2024 09:30
@Yamasaki-pan961
Copy link
Contributor Author

動作確認できました!
https://github.com/Yamasaki-pan961/use-template/commits/main/

@blendthink
Copy link
Member

@Yamasaki-pan961
もしかするとプライベートリポジトリになってるかもです、、!
ただ、無事に実行できたみたいで何よりです🥳

@Yamasaki-pan961
Copy link
Contributor Author

@blendthink
ああああ、公開しました😅

@blendthink
Copy link
Member

@Yamasaki-pan961
ありがとうございます!
こちらでも確認できました🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants