Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add vite example to ReadMe #44

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Add vite example to ReadMe #44

merged 1 commit into from
Nov 10, 2023

Conversation

pascalpp
Copy link

I struggled with getting Vite working in chalet and finally figured out that it needed the $PORT var included in the cmd as shown in some of the other examples. Since Vite is pretty popular these days I thought it was worth adding an explicit example in the ReadMe.

@@ -99,6 +99,7 @@ chalet add 'python -m SimpleHTTPServer $PORT' # static file server (Py
chalet add 'php -S 127.0.0.1:$PORT' # PHP
chalet add 'docker-compose up' # docker-compose
chalet add 'python manage.py runserver 127.0.0.1:$PORT' # Django
chalet add 'npm run dev -- --port $PORT' # Vite Dev Server
Copy link
Author

Choose a reason for hiding this comment

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

this could also be vite --port $PORT but I think most vite users would be using npm run dev. That requires the interim -- which might trip up some users as well, so helpful to have an example in this form.

@jeansaad jeansaad merged commit 03e377d into jeansaad:master Nov 10, 2023
@jeansaad
Copy link
Owner

Thank you for this PR!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants