-
Notifications
You must be signed in to change notification settings - Fork 24
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: Update dependencies and fix link_swift_library err #198
base: main
Are you sure you want to change the base?
Conversation
Haha, I just revert #196 because it didn't work with Tauri Specta 😅 |
But Tauri Specta is necessary for generate typesafe commands, do you know a way to make they work together? |
required by package tauri-specta v2.0.0-rc.10 the package tauri links to the native library Tauri, but it conflicts with a previous package which links to Tauri as well: |
Specta always update after few days to keep up with latest tauri beta. I think we can wait to update dependencies.
I've tried add |
If is there no way to make both work together, I think we can temporary remove Specta, because better error handling is necessary than typesafe commands at this time |
When I use cargo update, it doesn’t throw the type error, but it shows a link_swift_library error |
Perhaps we can wait for Specta to update |
Ah, you can use latest commit in |
Yes, I’ve already tried that |
Yes, I’ve encountered the same error, but I still don’t know how to fix it |
As far as I found, we need to update Error struct like this use specta::Type; <- Add specta type
#[derive(Debug, Error, Type)] <- Use specta type
pub enum Error {
#[error(transparent)]
Client(#[from] nostr_sdk::client::Error),
#[error(transparent)]
Key(#[from] nostr_sdk::key::Error),
#[error(transparent)]
Num(#[from] std::num::ParseIntError),
} Look like rust-nostr need to support Specta too. |
No description provided.