Skip to content
This repository has been archived by the owner on Nov 20, 2024. It is now read-only.

fix: [SPMVP-6809] support custom tailwind config for karbon cli #304

Conversation

SidStraw
Copy link
Contributor

@SidStraw SidStraw force-pushed the SPMVP-6809-cannot-execute-yarn-karbon-pack-only-in-briefstory-repo branch from 07ea16e to 51045ee Compare November 28, 2023 06:43
const packageJson = require(path.resolve(tempPath, './package.json'))
const version = packageJson?.packageManager?.split?.('@')?.[1] || 'berry'

await $$`yarn set version ${version}`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

這邊因為 yarn set version berry 會設置為 yarn 4.0
可能與專案設定相衝突而導致錯誤

darkMode: ['class', '.force-use-dark-mode'],
},
],
config: tailwindConfigPath,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

測試後發現 require(tailwindConfigPath) 沒有正確載入組態,因此改回使用 config: tailwindConfigPath 設定

@SidStraw SidStraw force-pushed the SPMVP-6809-cannot-execute-yarn-karbon-pack-only-in-briefstory-repo branch from 290f992 to 42f1d40 Compare November 28, 2023 07:01
@SidStraw SidStraw requested a review from DanSnow November 28, 2023 07:02
@DanSnow DanSnow changed the title fix: [SPMVP-6809] Cannot execute yarn karbon --pack-only in Briefstory repo fix: [SPMVP-6809] support custom tailwind config for karbon cli Nov 28, 2023
@DanSnow DanSnow merged commit 4579818 into main Nov 29, 2023
1 check passed
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.

2 participants