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

issueとpull requestのテンプレートの配置場所を修正 #4

Merged
merged 1 commit into from
Oct 1, 2021

Conversation

Tacha-S
Copy link
Contributor

@Tacha-S Tacha-S commented Oct 1, 2021

Summary

issue, pull requestテンプレートファイルを規定の配置場所へ移動

Test

  • 別リポジトリからこのリポジトリのissueテンプレートが参照される
  • 別リポジトリからこのリポジトリのpull requestテンプレートが参照される

@Tacha-S Tacha-S requested a review from h-wata October 1, 2021 03:55
@h-wata
Copy link
Contributor

h-wata commented Oct 1, 2021

今回の内容とは直接関係ないのですが、Pull Requestの方で、

catkin build`を通った

があると思うのですが、汎用的なTemplateだとすると少し文面は変えたほうがいいと思いました。
Testごと、コメントアウトしておいてもいいかもしれないです。

@Tacha-S
Copy link
Contributor Author

Tacha-S commented Oct 1, 2021

cubeリポジトリの方で追加されたMerge条件も入れてませんし、とりあえず既存のテンプレートリポジトリに存在するものをOrganization全体に反映させるまでにとどめています。

個人的には別issueとして、以下のあたりをアップデートするべきかなと思っています。

  • 指摘されているようにROSパッケージ用のテスト項目が入っていること
  • Merge条件が含まれていないこと
    • Merge条件も微妙な気がしますが...(依存するPRとかは項目としてあってもいいかと思いますが、誰のレビューがほしいとかは補足の項目でいいと思うので)
  • 解決されるissueをRefの見出しでなくfix #, close #, resolve #*で書かせるようになっていないこと

@h-wata
Copy link
Contributor

h-wata commented Oct 1, 2021

確かに別Issueがいいですかね。承知しました。

@h-wata h-wata merged commit 1350b7c into main Oct 1, 2021
@h-wata h-wata deleted the bugfix/template-path branch October 1, 2021 05:42
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.

issue, pull requestのテンプレートのディレクトリが間違っている
2 participants