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

Prettier v3 の全面的な導入と VSCode の設定 #503

Merged
merged 3 commits into from
Aug 13, 2024

Conversation

senkenn
Copy link
Contributor

@senkenn senkenn commented Aug 3, 2024

📑 Summary

  • Prettier が zenn-cli パッケージにしか使われていないようでしたので全てのパッケージで適応するようにした
  • .vscode/extensions.json で 推奨拡張機能の設定
  • .vscode/settings.json 各言語の設定
    • 各言語で指定しないと User settings.json のほうが有効になってしまうのでこのような設定にした
    • HTMLは packages/zenn-content-css/test.html での変更量が大きすぎたので設定を見送った
    • Markdown は画像のキャプション *foo bar*_foo bar_ に置き換わってしまったりと予期しない変更が入りそうだったので見送った
      (一応 _foo bar_ でも正しくプレビューできているようではあるが、、)
  • Prettier 設定類のコミットは e7f159f, その設定でフォーマットしたコミットを f15a55a に分けた
  • Prettierのバージョンを上げた:2.8.4 -> 3.3.3

Resolves #501

📋 Tasks

プルリクエストを作成いただく際、お手数ですが以下の内容についてご確認をお願いします。

  • 📖 Contribution Guide を読んだ
  • 👩‍💻 canary ブランチに対するプルリクエストである
  • zenn-cli で実行して正しく動作しているか確認する
  • 不要なコードが含まれていないか( コメントやログの消し忘れに注意 )
  • XSS になるようなコードが含まれていないか
  • モバイル端末での表示が考慮されているか
  • Pull Request の内容は妥当か( 膨らみすぎてないか )

より詳しい内容は Pull Request Policy を参照してください。

@senkenn senkenn changed the title Prettier の全面的な導入to Prettier の全面的な導入と VSCode の設定 Aug 3, 2024
@senkenn
Copy link
Contributor Author

senkenn commented Aug 3, 2024

このPRには CI でのチェックが含まれていませんが、それも実装したほうが良ければ実装します。

@senkenn senkenn changed the title Prettier の全面的な導入と VSCode の設定 Prettier v3 の全面的な導入と VSCode の設定 Aug 5, 2024
@cm-wada-yusuke cm-wada-yusuke added the enhancement 機能以外の改善 label Aug 7, 2024
Copy link
Member

@cm-igarashi-ryosuke cm-igarashi-ryosuke left a comment

Choose a reason for hiding this comment

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

ご対応ありがとうございます!貢献に感謝します!

このPRの内容についてはLGTMです!他のPRをマージすることで、いくつかコンフリクトが発生すると思いますのでそこだけご対応をお願いします 🙇

その他、今後対応したいことを別のissueにしておきました。もし関心ありましたら引き続きご対応いただけると嬉しいです。
#506

@senkenn
Copy link
Contributor Author

senkenn commented Aug 13, 2024

@cm-igarashi-ryosuke コンフリクト解消しました!

Copy link
Member

@cm-igarashi-ryosuke cm-igarashi-ryosuke left a comment

Choose a reason for hiding this comment

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

修正ありがとうございました!
LGTMですのでマージします!

@cm-igarashi-ryosuke cm-igarashi-ryosuke merged commit 3d8acd1 into zenn-dev:canary Aug 13, 2024
3 checks passed
@senkenn senkenn deleted the senkenn/issue#501 branch August 13, 2024 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 機能以外の改善
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[提案]: Prettier の全面的な導入
3 participants