Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

nuxt.config.ts で @fullhuman/postcss-purgecss をimport文でインポートする #5457

Closed
wants to merge 1 commit into from
Closed

nuxt.config.ts で @fullhuman/postcss-purgecss をimport文でインポートする #5457

wants to merge 1 commit into from

Conversation

munierujp
Copy link
Contributor

👏 解決する issue / Resolved Issues

⛏ 変更内容 / Details of Changes

  • nuxt.config.tspurgecss に型をつけるために @fullhuman/postcss-purgecss のインポートを require() からimport文に変更
  • 記載位置が変更されているのはESLintの simple-import-sort/sort ルールによるものです

@kaizumaki
Copy link
Collaborator

@munierujp ご提案ありがとうございます!いま #5432 と、それに対するPR #5443 が上がっており、こちらのPRと内容がコンフリクトすると思われます。どうするのがいいのでしょう...? これらのissue, PRのほうも @munierujp さんのご意見伺いたく思います 🙏

@munierujp
Copy link
Contributor Author

@kaizumaki #5443 によって purgecss のインポート自体がなくなるので、 #5443 がマージされ次第 #5456, #5457 を(マージせずに)クローズできます。

@munierujp
Copy link
Contributor Author

munierujp commented Oct 1, 2020

#5443 によって purgecss のインポート自体が無くなったので close します。

@munierujp munierujp closed this Oct 1, 2020
@munierujp munierujp deleted the update_nuxt_config_puregecss branch October 1, 2020 10:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nuxt.config.ts で @fullhuman/postcss-purgecss が require() でインポートされている
2 participants