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

README: remove deprecated plugins, add craco-esbuild #434

Merged
merged 1 commit into from
Jul 2, 2022

Conversation

leothorp
Copy link
Contributor

@leothorp leothorp commented Jul 2, 2022

I've been finding the craco-esbuild plugin useful, figured it would be good to have in the README.

I also removed a couple whose READMEs indicate they are deprecated. if you think it's better to keep them there and just mark [deprecated], I could change it to that as well. Thanks!

@dilanx
Copy link
Owner

dilanx commented Jul 2, 2022

Thanks for the PR @leothorp. I plan on moving most of that info to a new documentation system but it'll definitely be good to have it here for now and to have those same plugins in the new docs.

If you could keep the old ones but mark them as deprecated, that would be great!

@dilanx
Copy link
Owner

dilanx commented Jul 2, 2022

Actually, never mind. I just looked at the deprecated plugins and it looks like the alternatives are either officially supported or already on the list. What you have is good.

@dilanx dilanx merged commit 93d9507 into dilanx:master Jul 2, 2022
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.

2 participants