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

feat!: stylelint v14 を peerDependencies から除外、v15 のルールに移行 #205

Merged
merged 2 commits into from
May 12, 2023

Conversation

f440
Copy link
Member

@f440 f440 commented May 9, 2023

stylelint v14 から v15 で大きく方針変更が発生しており、stylelint v15 系を利用すると peerDependencies を満たすことができなくなっています。

stylelint v14 系のサポートをやめ、ドキュメントをもとに v15 系の設定に移行しました。 cf. Migrating to 15.0.0 | Stylelint

  • stylelint v14 を peerDependencies から除外
  • warning が出るルールを削除
  • postcss-jsx から postcss-styled-syntax への移行

本PRは非互換な変更になります。また、すでに導入済みのプロジェクトでは以下の対応を行う必要があります。

  • package.json の変更
    • stylelint-config-prettier を削除
    • postcss-jsx を削除
    • postcss-styled-syntax を追加

cf. [Migrating to 15.0.0](https://stylelint.io/migration-guide/to-15/)

- stylelint v14 を peerDependencies から除外
- warning が出るルールを削除
- postcss-jsx から postcss-styled-syntax への移行
@f440 f440 requested a review from a team as a code owner May 9, 2023 13:12
@f440 f440 requested review from nabeliwo, TakahiroHimi and diescake and removed request for a team May 9, 2023 13:12
Copy link
Member

@nabeliwo nabeliwo left a comment

Choose a reason for hiding this comment

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

対応ありがとうございます!LGTM 🎉

@diescake diescake changed the title stylelint v14 を peerDependencies から除外、v15 のルールに移行 feat!: stylelint v14 を peerDependencies から除外、v15 のルールに移行 May 10, 2023
README.md Outdated Show resolved Hide resolved
@diescake
Copy link
Contributor

diescake commented May 10, 2023

@f440

PRありがとうございます!!
1点軽微なコメントしたので確認お願いします!!!

あと、BREAKING CHANGESを含むのでタイトルにconventional commitsの作法でタイトルに feat!: を加えちゃいました!🙏
念のため、マージのときにコミットメッセージに入っているか確認お願いします〜

Copy link
Contributor

@diescake diescake left a comment

Choose a reason for hiding this comment

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

軽微な指摘なので再レビュー不要です!GoGo〜

@f440 f440 merged commit 5e87717 into kufu:master May 12, 2023
@f440
Copy link
Member Author

f440 commented May 12, 2023

確認&指摘どもです!マージしました!

@f440 f440 deleted the stylelint-v15 branch May 12, 2023 04:04
@f440 f440 restored the stylelint-v15 branch May 12, 2023 04:04
@f440 f440 deleted the stylelint-v15 branch May 12, 2023 04:04
@f440 f440 restored the stylelint-v15 branch May 12, 2023 04:04
],
customSyntax: 'postcss-jsx',
Copy link
Member Author

@f440 f440 May 12, 2023

Choose a reason for hiding this comment

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

  1. postcss-jsx が生まれる
  2. 更新が止まる
  3. stylelintがpostcss-css-in-jsとしてフォークする
  4. postcss-css-in-jsもアーカイブされ、乗り換えを促される

という流れを踏まえて変更しています。

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 this pull request may close these issues.

3 participants