feat: add tailwind support#416

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Closed

Yordan-Ramchev wants to merge3 commits intovuejs:mainfromYordan-Ramchev:add-tailwind-option

Closed

feat: add tailwind support#416

Yordan-Ramchev wants to merge3 commits intovuejs:mainfromYordan-Ramchev:add-tailwind-option

Conversation

Copy link

Contributor

Yordan-Ramchev commentedDec 30, 2023•
edited
Loading

Hi@sodatea @cexbrayat,

I've opened a pull request(re#246) that introduces:

Introduced Tailwind prompt functionality with an untranslated prompt message.
Ensured compatibility for both JavaScript (js) and TypeScript (ts) scenarios.
Modified generateReadme.ts for necessary updates.
Incorporated the Tailwind feature flag into snapshot.mjs.

I would appreciate your time and expertise in reviewing these changes.

Thank you!

feat: add tailwind support

fce7ce9

Copy link

bmulholland commentedDec 30, 2023

One concern is that the approach athttps://github.com/vuejs/create-vue-templates isn! on the number of options. Probably also the CI matrices? There's already a lot of directories in that repo, and over 100 CI checks... at what point will that approach break down?

I recognize that this is not about your code itself, but about the practicalities of maintaining all combinations of options.

At a minimum, you probably need to add the tailwind option tohttps://github.com/vuejs/create-vue/blob/main/.github/workflows/ci.yml