Skip to content
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 immediate dependencies #643

Merged
merged 3 commits into from
Oct 7, 2021
Merged

update immediate dependencies #643

merged 3 commits into from
Oct 7, 2021

Conversation

clux
Copy link
Member

@clux clux commented Oct 7, 2021

cycled this until the outdated list was empty

cargo outdated -d 1
cargo upgrade somecrate --all

and the list was quite large so giving this a try in a pr.
probably means we have a gap here to get automatic prs for dependency updates

cycle
```sh
cargo outdated -d 1
cargo upgrade somecrate --all
```

until empty list

Signed-off-by: clux <[email protected]>
@clux
Copy link
Member Author

clux commented Oct 7, 2021

rustls, pem and webpki has breaking changes, opening a separate issue to tackle that upgrade separately.

@clux clux added the dependencies upgrades to dependencies label Oct 7, 2021
@kazk
Copy link
Member

kazk commented Oct 7, 2021

probably means we have a gap here to get automatic prs for dependency updates

Are you trying to figure out why Dependabot hasn't opened PRs? https://github.com/kube-rs/kube-rs/network/updates has status and look like there's an error.

image

error: failed to parse manifest at `/home/dependabot/dependabot-updater/dependabot_tmp_dir/examples/Cargo.toml`

Caused by:
  the binary target name `examples` is forbidden, it conflicts with with cargo's build directory names

@clux
Copy link
Member Author

clux commented Oct 7, 2021

ah, yeah, i was trying to figure that out. weird error.
our setup is the same as tokio's examples, but it seems they also rejected dependabot for completely understandable reasons in tokio-rs/tokio#3744 so maybe it's good to just run this periodically before breaking version bumps.

@clux clux merged commit 65bd0bd into master Oct 7, 2021
@clux clux deleted the bump-deps branch October 7, 2021 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies upgrades to dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants