Update Specta in lockfile to fix documentation#15177
Update Specta in lockfile to fix documentation#15177oscartbeaumont wants to merge 4 commits intotauri-apps:devfrom
Conversation
Package Changes Through 83ebd65There are 9 changes which include tauri-macos-sign with patch, tauri-build with patch, tauri with minor, tauri-bundler with minor, tauri-cli with minor, @tauri-apps/cli with minor, tauri-runtime-wry with minor, tauri-runtime with minor, tauri-utils with minor Planned Package VersionsThe following package releases are the planned based on the context of changes in this pull request.
Add another change file through the GitHub UI by following this link. Read about change files or the docs at github.com/jbolda/covector |
|
I have extended this PR with a couple of other dependency upgrades to avoid using dependencies with It looks like These issues caused the Tauri Specta docs to fail to build in the latest release so be aware they may also cause issues for Tauri's next release. |
The docs.rs build for Tauri 2.10.3 failed.
In looking into the build logs this seems to be due to a bug in the Specta.
The type implementation for
[T]looked like the following:and then the implementation of
Vec<T>is the following:This is an issue because the implementation for
Vec<T>is not available without the newstdfeature (which is enabled by default) breaking the implementation for[T]which shouldn't require thestdfeature. Thestdfeature being enabled by default meant that it was never caught in Specta's tests, so I will need to upgrade the CI workflows! As Tauri disables default features thestdfeature is missing when it builds docs and this code fails to compile. In real-world applications using Tauri + Specta I doubt this will be encountered as thespectadependency being in the usersCargo.tomlwill likely enable thestdfeature.Prior to me becoming aware of this being an issue in Tauri I had already overhaulled all the types impls so now
Vec<T>dispatches to[T]which is how is how it should have always been to begin with. This PR just bumps the version to the latest release I did today to ensure we can get this issue resolved as quick as possible.Really sorry for this breakage. Once this PR is merged we should be able to push a minor release again so the docs can be published!