Skip to content
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

docs: add awesome-vite link in Backend Integrations section #1586

Merged
merged 2 commits into from
Mar 15, 2021

Conversation

ElMassimo
Copy link
Contributor

@ElMassimo ElMassimo commented Jan 19, 2021

Description 📖

This pull request mentions awesome-vite in the Backend Integration section, so that users looking to integrate Vite in Laravel and Rails projects can do so more easily.

Copy link
Member

@antfu antfu left a comment

Choose a reason for hiding this comment

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

Thank you @ElMassimo! But after some discussions with the team, we think maybe it's better to keep them in the awesome-lite for now.

If you don't mind, I also think it's good to mention https://github.com/vitejs/awesome-vite#integrations-with-backends in this section instead.

@ElMassimo ElMassimo changed the title Add Vite Rails to the Backend Integration section Add awesome-vite link in Backend Integrations section Mar 15, 2021
@ElMassimo
Copy link
Contributor Author

Makes sense, Awesome Vite should be easier to keep up-to-date, and that way there are less pull requests to this repo.

@antfu I updated the PR with the suggested changes, thanks.

Co-authored-by: Anthony Fu <anthonyfu117@hotmail.com>
@antfu antfu changed the title Add awesome-vite link in Backend Integrations section docs: add awesome-vite link in Backend Integrations section Mar 15, 2021
@antfu antfu merged commit 6d5c9c6 into vitejs:main Mar 15, 2021
@ElMassimo ElMassimo deleted the patch-1 branch March 15, 2021 14:55
This was referenced Mar 15, 2021
This was referenced Mar 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants