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

fix(yew-notifications): Update yew to latest version (0.21.0) #6

Closed
wants to merge 2 commits into from
Closed

Conversation

orangethewell
Copy link

This pull request fixes compatibility with latest yew version and increments minor version of the crate in 1.

@TheBestTvarynka
Copy link
Owner

Hi
Thank you for your contribution. Please, fix formatting 🥺

I'll have a look at it today (around the evening). I wanna perform a few tests

@orangethewell
Copy link
Author

No problem, I'll fix it. I just don't know exactly what the problem is, is it because there is a line break? I have never used cargo-fmt

@TheBestTvarynka
Copy link
Owner

just run cargo +nightly fmt --all in the terminal. It will do everything we need. If you get an error, then try to run rustup toolchain install nightly

@TheBestTvarynka
Copy link
Owner

relaced with #7

I'm closing this PR for now. But thank you for your attention to this project anyway. I decided to do it by myself because I also need to fix the code and update other dependencies

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants