Bump time version from 0.3.6 to 0.3.44 to bump deranged version from 0.4.0 to 0.5.x#382
Bump time version from 0.3.6 to 0.3.44 to bump deranged version from 0.4.0 to 0.5.x#382iamjpotts wants to merge 1 commit into
Conversation
…0.4.0 to 0.5.x Signed-off-by: Joshua Potts <8704475+iamjpotts@users.noreply.github.com>
|
I don't think we need to bump the requirement for this, only the lockfile. |
|
It still failed the same way after the bump. I'm digging into it more. I have an unconfirmed suspicion that a change to nightly caused this. |
|
This command: Succeeds with |
|
Related: ratatui/ratatui#2102 PR linked in error text: I am able to reproduce the error by cloning the https://github.com/jhpratt/deranged crate and running |
|
I opened a pr on the dependency crate: jhpratt/deranged#24 Bumping the In the meantime before a fix to |
Sounds good, thank you for looking into this! |
|
Is it even possible to pin the version of nightly that docs.rs uses? |
FYI MSRV bumps are usually not treated as semver-breaking in the ecosystem. But, I don't think we need to do an MSRV bump here -- we can just stick to an older time/deranged in our lockfile for now. |
It can be pinned in the github workflow file. |
But I understood the proposed fix to be about pinning it in our CI so the docs workflow can do its thing. |
|
But isn't the docs workflow's purpose to demonstrate that the docs.rs build will work once the crate is published? |
|
I think the right thing to do here is rename Also worth noting that there is a missing |
Ah yeah, that's fair. |
|
CI failure that prompted this PR was resolved by #384. |
Resolves new ci failure unrelated to a code change: