-
Notifications
You must be signed in to change notification settings - Fork 0
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
chore: explicitly describe RatingLoadingIcon size when rendering it #276
chore: explicitly describe RatingLoadingIcon size when rendering it #276
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@FrancoAguzzi Changes here look good. Can you please prepare a changeset as part of this PR? This can be a nice practice opportunity for us.
@lightwalker-eth I'd love to but I can't! 😃⬇️ Changeset is not tied to the In this PR I only changed one file from within Image below reflects what we see in terminal after a Indeed I want to practice Changeset usage! No doubts I will soon be able to do it in some of our PRs 🚀🙌🏼 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@FrancoAguzzi You're right! Thanks for your good comment. Reviewed and approved 👍
No description provided.