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.

Sign up for GitHub

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

Jump to bottom

@next refers old version (v4.5.3) #1454

Merged
merged 1 commit into from
May 19, 2020
Merged

@next refers old version (v4.5.3) #1454

merged 1 commit into from
May 19, 2020

Conversation

mtsmfm
Copy link
Contributor

@mtsmfm mtsmfm commented May 19, 2020 •
edited
Loading

@theKashey theKashey merged commit 3b61e33 into gaearon:gh-pages May 19, 2020
@theKashey
Copy link
Collaborator

👍

@mtsmfm
Copy link
Contributor Author

mtsmfm commented May 20, 2020

@theKashey next is currently not next and beta is not beta I think.
Do you have any plans to remove these tags?

@mtsmfm mtsmfm deleted the no-next branch May 20, 2020 03:50
@theKashey
Copy link
Collaborator

There is a plan to remove RHL :) Not joking - this package is very close to be deprecated.

@mtsmfm
Copy link
Contributor Author

mtsmfm commented May 20, 2020

I know this issue #1453 though,
I think react-refresh is still experimental and we'll need some months to deprecate.

@theKashey
Copy link
Collaborator

It's already used in nextjs and react-native, and even RHL uses it from some point of view.

@mtsmfm
Copy link
Contributor Author

mtsmfm commented May 20, 2020

I see.

Anyway, this package is not deprecated yet, removing old tags can avoid confusing from now I think.

https://docs.npmjs.com/cli/dist-tag

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mtsmfm @theKashey