cpprefjp / site

cpprefjpサイトのMarkdownソース
https://cpprefjp.github.io/
382 stars 157 forks source link

outer link check failed at c12a9d70fa976a36dd29402c6cf4d6385e0537c6 #869

Closed github-actions[bot] closed 3 years ago

github-actions[bot] commented 3 years ago

The commit hash was: c12a9d70fa976a36dd29402c6cf4d6385e0537c6.

URL https://github.com/melpon/qiita/tree/master/items/stdstringのSSO(Small-string%20optimization) not found. 404 from:['lang/cpp20/changing_the_active_member_of_a_union_inside_constexpr.md']

yohhoy commented 3 years ago

リンクURLが途中で切れてしまっている? https://github.com/melpon/qiita/tree/master/items/stdstring%E3%81%AESSO(Small-string%20optimization)%E3%81%8C%E3%81%A9%E3%81%86%E3%81%AA%E3%81%A3%E3%81%A6%E3%81%84%E3%82%8B%E3%81%8B%E8%AA%BF%E3%81%B9%E3%81%9F

yumetodo commented 3 years ago
- [std::stringのSSO(Small-string optimization)がどうなっているか調べた - melpon/qiita](https://github.com/melpon/qiita/tree/master/items/stdstringのSSO(Small-string%20optimization)がどうなっているか調べた)

むしろこれ有効なMarkdownなんですね

onihusube commented 3 years ago

あんまり気にしてなかったですが、URLエンコードしたといたほうが良さげでしたか・・・

faithandbrave commented 3 years ago

それか、URLの抽出をする正規表現を直すかですね…。カッコを含むURLはむずかしい…。 https://github.com/cpprefjp/site/blob/master/.github/workflows/script/link_check.py#L51

yumetodo commented 3 years ago

https://abicky.net/2012/08/04/181414/ こういうことですかね

faithandbrave commented 3 years ago

そういうのです。

faithandbrave commented 3 years ago

markdownライブラリでパースするのがかんたんそうです。 https://stackoverflow.com/questions/30734682/extracting-url-and-anchor-text-from-markdown-using-python

faithandbrave commented 3 years ago

これはmarkdown -> html -> xmlと変換してから解析するので、すごく重そう。

faithandbrave commented 3 years ago

パースしたURLに開きカッコが含まれていたら、2個目の閉じカッコを検索するようパッチを当てようかと思います…。

正規表現で単にvalidなURLをパースしてもうまくいかないような気がしていて、Markdown記法としてのURLをパースしないといけなくてむずかしいので。

faithandbrave commented 3 years ago

CI通りました。