- Notifications
You must be signed in to change notification settings - Fork328
update dependencies for dashboard#1108
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| rocket = { git = "https://github.com/SergioBenitez/Rocket", features = ["secrets", "json"] } | ||
| sailfish = "0.8" | ||
| sailfish = "0.8.0" # 0.8.1 has breaking changes |
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.
What's the breaking change?
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.
Templates no longer compile, due to missing min/max args. It's not clear why, since those variables are not declared by our templates.
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.
Just need to clean the template cache.cargo clean
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.
I think you may be right. I filed the issue with the authors:rust-sailfish/sailfish#137
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.
Easy fix:rust-sailfish/sailfish#138
Debugging is actually pretty fun!
No description provided.