-
Notifications
You must be signed in to change notification settings - Fork 40
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
Update Rust crate tough to 0.15 #4477
Conversation
c7b57cb
to
96d749c
Compare
96d749c
to
768dff6
Compare
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. ⚠ Warning: custom changes will be lost. |
Ah crap, some failures that I need to address. |
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.
Thanks for going through this - looks pretty tedious, and I apologize for some of the additionally-tedious comments I left. :-/
.map_err(RepositoryError::LoadRepository)?; | ||
|
||
let artifacts = repository | ||
.read_artifacts() | ||
.await | ||
.map_err(RepositoryError::ReadArtifactsDocument)?; | ||
|
||
// Create another temporary directory where we'll "permanently" (as long |
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.
UpdatePlanBuilder::new()
uses camino_tempfile
to create a temporary directory - should we spawn_blocking
it? (Feels like maybe "technically yes since it's I/O, but practically probably doesn't matter"?)
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.
Ehhhh I really don't think it matters to be honest. This isn't an operation that takes a long time.
This PR contains the following updates:
0.14
->0.15
Release Notes
awslabs/tough (tough)
v0.15.0
Compare Source
Changes
Configuration
📅 Schedule: Branch creation - "after 8pm,before 6am" in timezone America/Los_Angeles, Automerge - "after 8pm,before 6am" in timezone America/Los_Angeles.
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Renovate Bot.