kagan

Results 21 comments of kagan

@YusukeHirao このPRでwebsiteのビジュアルリグレッションテストを追加したいのですが、画像の保存のための権限がなく失敗しているようです。 https://github.com/markuplint/markuplint/actions/runs/7512820384/job/20453916346?pr=1386 workflowのパーミッションを追加してもらうことは可能でしょうか? `Settings > Actions > General > Workflow permissions` を `Read and write permissions` に変更していただきたいです。 参考: https://zenn.dev/tatsugon/articles/github-actions-permission-error

@YusukeHirao パーミッション変更ありがとうございました! その後、`pull_request` トリガーだとビジュアルリグレッションテストの結果をコメントするための権限がなく、エラーになることがわかりました。 `pull_request_target` トリガーで実行させる必要があり、そのように変更しています。 `pull_request_target` トリガーのワークフローはマージ後にしか実行されないため検証ができていないのですが、マージされてから動作確認しようと思います。 `pull_request_target` トリガーはSecretsにアクセスできるなどセキュリティ上の問題があるため、ラベルをつけたときのみ実行するようにしています。 https://pankona.github.io/blog/2021/03/29/github-actions-pull-request-target/ https://securitylab.github.com/research/github-actions-preventing-pwn-requests/ ご確認お願いします!

@YusukeHirao That's a good alternative! Alias naming would make it easier to understand what the rule means. Also, the slash-separated style is similar to [ESLint's plugin rule naming](https://eslint.org/docs/latest/user-guide/configuring/plugins#use-a-plugin), so it...

反応遅くなりました 🙏 不整合どうにかしないとなので解消方法考えます…! >meta.categoryをschema.json持っていく工夫が必要かもしれません。 websiteのビルドに必要な情報としてはたしかにschema.jsonに寄せたほうが楽ではあるのですが、以前 [markuplint本体の依存パッケージが大きい問題](https://github.com/markuplint/markuplint/issues/1336) の解消のために [schema.jsonからindex.ts に移動した](https://github.com/markuplint/markuplint/pull/1335) 経緯があるので、これを戻すのはあまりしたくないのですよね… なのでwebsiteのためにschema.jsonに記述する場合、index.tsと二重管理になってしまうかなと思っています 他のライブラリでの例も見てみて、方法考えます

@YusukeHirao Thank you for your idea! I'm glad that I have an opportunity to reconsider it. My opinions: Users (including me) want to know **"What codes does this rule detect?"**...

@YusukeHirao 新ロゴもできたことなので、vscode-iconsへの追加を検討しませんか? 👀 ロゴの画像ファイルと、適用するファイル名などの設定を追加してPRを提出すれば、わりとすぐにマージしてもらえるようです。 例: https://github.com/vscode-icons/vscode-icons/pull/3151 なお、ライトテーマ用の画像の追加は任意です。 jestのように、白縁取りをつけることで1つの画像でどちらでも見やすいように対応しているアイコンもあります。 https://github.com/vscode-icons/vscode-icons/blob/master/icons/file_type_jest.svg ちなみに、vscode-iconsの設定はmarkuplintのVSCode拡張機能とは独立した問題ではあるので、markuplint本体のissueにしておくほうが自然かなと思います。 (とはいえ、VSCode関連のissueをこちらにまとめるわかりやすさもあると思うので、どちらでも問題ないです!)

vscode-iconsへの追加、私の方で対応しますね

There is a static "404 page" (`404.html`). So it seems natural that /404 returns 200 response. If it should return a 404 status code, we would need to configure the...

@YusukeHirao 早速ご準備ありがとうございます! リポジトリ名について、 `markuplint-playground` にしておきたいのですがいかがでしょう? markuplintのorganization配下にあるので自明ではあると思いつつ、フォークしたときなど、リポジトリ名単体で表示されるときのことを考えるとリポジトリ名に `markuplint-` を含めておきたいです。 移行作業は追って私の方で対応しますね。

@YusukeHirao markuplint-playground リポジトリにファイルを追加し、こちらのリポジトリからデプロイされるようにNetlifyの設定を変更しました! デプロイ後の動作も確認済みです。 https://playground.markuplint.dev 後ほど、markuplint(本体)リポジトリからplaygroundのファイルを削除するPRを作成します。