Skip to content

GLOBAL_QUALIFY_LISTにコメントを書けるようにした #91

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 14, 2025
Merged

GLOBAL_QUALIFY_LISTにコメントを書けるようにした #91

merged 1 commit into from
Jul 14, 2025
+6 −1

Conversation

faithandbrave
Copy link
Member

cpprefjp/site#1452 (comment)

GLOBAL_QUALIFY_LIST.txtですが、コメントでルールを書けるようにしたいです。
スペースを取り除いた行頭#をコメントとして扱おうかと思いますが、これに異論ある方はいますでしょうか？

ここになにを書くかは継続議論しましょう。
ひとまずツールを整備をしたいです。

This was referenced Jul 8, 2025
Open
Open
@akinomyoga
Copy link
Member

パターンに一致しなければ無視するようになっているので、既にコメントは書ける気がするのですが、敢えて site_generator 側で処理を追加する必要はありますか?

或いは、この変更と併せて ValueError を握りつぶさないように変更して、新たに「コメントと実際の指定以外の行を明示的にエラーにする」というように、制限を厳しくするという方向の変更もありだとは思います。

@faithandbrave
Copy link
Member Author

フリーフォーマットよりはルールとして決まっていた方が、後々さらに構文を追加する際にもうれしい気はしますね

@faithandbrave
Copy link
Member Author

マージします！

@faithandbrave faithandbrave merged commit a5ea672 into master Jul 14, 2025
@faithandbrave faithandbrave deleted the comment_syntax_for_global_qualify branch July 14, 2025 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yumetodo yumetodo yumetodo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@faithandbrave @akinomyoga @yumetodo