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

doc: use a more friendly link to codemod tool #3266

Merged
merged 1 commit into from May 12, 2023

Conversation

HerringtonDarkholme
Copy link
Contributor

Description

Hi yew team, I’m the author of the ast-grep, the tool you mentioned in migration guide.

You have done an amazing job with yew! It is one of the most innovative and impressive Rust web frameworks I have ever seen. I’m thrilled to see that ast-grep could help you. It’s my very design goal to make it easier for OSS teams to manage API changes and code migration and I’m glad you find it useful. Really appreciate your support and trust! 🥰

I noticed that you are linking to the crates.io page of the ast-grep in your README.md file. While this is a valid link, I would like to suggest changing the link to the official website of ast-grep. The official website includes a more detailed description of the tool, as well as installation instructions and usage examples. The crates.io page is more suitable for developers who want to install or depend on ast-grep as a library. It also looks less professional and trustworthy than the official website to the end users.

I hope this PR will help yew users to migrate the framework with fewer concerns. Thank you for your time and consideration!

Checklist

  • I have reviewed my own code
  • I have added tests

Copy link
Member

@futursolo futursolo 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 for the pull request!

The pending GitHub Actions is stuck should not be related to the content of this pull request but an GitHub outage happened earlier today.

Copy link
Member

@WorldSEnder WorldSEnder left a comment

Choose a reason for hiding this comment

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

o/ Thanks for the feedback, I appreciate it.
Lgtm

@HerringtonDarkholme
Copy link
Contributor Author

Thank Yew team! Let me know if you need anything from me to retry the GH action! 😃

@WorldSEnder
Copy link
Member

I guess the deployment of the website preview has decided to run forever. In any case, the PR is simple enough for me to merge as-is without further feedback from the pre-deploy :)

@WorldSEnder WorldSEnder merged commit 2c4ea60 into yewstack:master May 12, 2023
3 of 4 checks passed
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.

None yet

3 participants