Skip to content
This repository has been archived by the owner on Jan 5, 2025. It is now read-only.

Update issue templates : GLOSSARY_proposal, TRANSLATION_error, OTHER #36

Merged
merged 4 commits into from
Jun 9, 2024

Conversation

aconite-ac
Copy link
Member

@aconite-ac aconite-ac commented Jun 8, 2024

  • 対訳表への変更提案
  • 翻訳の誤りの指摘
  • そのほか

に対応する issue template を作成

* 対訳表への変更提案
* 翻訳の誤りの指摘
* そのほか
に対応する issue template を作成
@aconite-ac aconite-ac requested a review from Seasawher June 8, 2024 14:36
@aconite-ac
Copy link
Member Author

@Seasawher
#35 のご提案を受けて issue template を3種類作成しました。
review のほどよろしくお願いいたします。

@Seasawher
Copy link
Member

Seasawher commented Jun 8, 2024

https://github.com/Seasawher/fp-lean-ja/issues/new/choose

確認ですが,上記のような見た目になります.

@aconite-ac
Copy link
Member Author

@Seasawher
フォークを用いたプレビュー、ありがとうございます。
個人的には大丈夫そうに見えますが、
慣例から外れている箇所や、修正案などございましたらご指摘いただけるとありがたいです。
(commit して頂いても大丈夫です。)

@aconite-ac
Copy link
Member Author

GitHub上で issue template を作成した際に、name がファイル名に反映されたこと、また nameabout で同語反復が起きることを鑑みてこのような name にしました。
…が、about をより具体的に書いた上で name を日本語にリネームした方がユーザーフレンドリーですね。

@@ -0,0 +1,10 @@
---
Copy link
Member

Choose a reason for hiding this comment

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

勘違いしていたのですが,テンプレートにない issue を投稿する機能がデフォルトで存在するため,others というテンプレートは必要がないようです.このファイルは削除しましょう.

Copy link
Member Author

Choose a reason for hiding this comment

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

確かに issue template を選ぶ画面の下部の「Open a blank issue.」をクリックすると空の issue を開けますね。
ファイルを削除する commit まで出していただきありがとうございます。

@aconite-ac
Copy link
Member Author

@Seasawher
修正commit・review・approve ありがとうございます!
未解決の問題はないと思われますので、merge いたします。

@aconite-ac aconite-ac merged commit 858802c into main Jun 9, 2024
2 checks passed
@aconite-ac aconite-ac deleted the issue-template-1 branch June 9, 2024 12:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants