-
-
Notifications
You must be signed in to change notification settings - Fork 263
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(create-analog): add tailwind gen to create-analog script #315
feat(create-analog): add tailwind gen to create-analog script #315
Conversation
✅ Deploy Preview for analog-blog ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for analog-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for analog-app ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@brandonroberts @goetzrobin please take a look 👀 |
@@ -0,0 +1,8 @@ | |||
/** @type {import('tailwindcss').Config} */ | |||
module.exports = { | |||
content: ['./index.html', './src/**/*.{html,ts}'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add .md
for content routes
content: ['./index.html', './src/**/*.{html,ts}'], | |
content: ['./index.html', './src/**/*.{html,ts,md}'], |
packages/create-analog/index.js
Outdated
} | ||
|
||
function addDevDependencies(pkg) { | ||
['tailwindcss', 'postcss', 'autoprefixer'].forEach( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There should be specific versions for these. Let's use the current versions with a ^
on the front
packages/create-analog/package.json
Outdated
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "create-analog", | |||
"version": "0.2.9", | |||
"version": "0.3.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can bump the last version, as the middle one is considered the major because the first one is a 0
according to semver
"version": "0.3.0", | |
"version": "0.2.10", |
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
User add tailwind to existing template via prompt or flag.
Which package are you modifying?
What is the current behavior?
Closes #303
What is the new behavior?
Does this PR introduce a breaking change?
Other information