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

ヘッダファイルの表記ゆれ #2395

Closed
koizumistr opened this issue Sep 26, 2022 · 7 comments · Fixed by #2606
Closed

ヘッダファイルの表記ゆれ #2395

koizumistr opened this issue Sep 26, 2022 · 7 comments · Fixed by #2606

Comments

@koizumistr
Copy link
Contributor

重要度

問題点

ヘッダファイル、ヘッダーファイルと表記がゆれています(ヘッダファイルの方が多数派)

背景

#2358 の確認の過程で見つけました。(コメントとして書いてしまったので埋もれてしまうのでこちらにしました。)

解決方法

特別な理由がなければ多数派のヘッダファイルに合わせるべきかと思います。

注意点

No response

貢献者として記載可否

以前の記載名を使用

貢献者名

No response

@koizumistr
Copy link
Contributor Author

#2280 とちょっと関係あるでしょうか。https://atmarkit.itmedia.co.jp/news/200807/25/microsoft.html をみると header は「ヘッダー」になってますね。が、ここでは「ヘッダファイル」と組み合わせた場合についてだけ言っているつもりです。

@KenichiroTanaka
Copy link
Contributor

KenichiroTanaka commented Sep 29, 2022

個人的にはなかなかヘッダーファイルという表記には慣れないです。
ひとまずヘッダファイルで良いように思いますが、皆様どうでしょうか。
@noborus さん
@tatsuo-ishii さん
@harukat さん
@kitayama さん

@noborus
Copy link
Contributor

noborus commented Sep 29, 2022

ヘッダファイルはヘッダファイルのままで良いと思います。
#2280 は他では長音を付けるルールの単語をこちらでもそのルールにする話で、全体の長音付けルールの変更はその後です。
今は統一するのを優先すれば良いと思います。

@tatsuo-ishii
Copy link
Contributor

私はどちらに統一するのでも良いです。

@KenichiroTanaka
Copy link
Contributor

コメントありがとうございます。
では、本issuesの対応としては「ヘッダファイルに統一する」で進められればと思います。

@koizumistr
Copy link
Contributor Author

.ng.list に「ヘッダーファイル」が追加されたのでこちらはクローズ、
と思ったですが、ファイルに追記されたもののまだコメントアウト、という状態でしたね、、、

@noborus
Copy link
Contributor

noborus commented Oct 31, 2022

まだbki.sgmlにヘッダーファイルが残っているので、そちらを変更してからng.listを有効にする必要があります。

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 a pull request may close this issue.

4 participants