Skip to content

Use resolv dependency from crates.io#40

Merged
lvkv merged 4 commits intomainfrom
lvkv/resolv-dep
Apr 16, 2026
Merged

Use resolv dependency from crates.io#40
lvkv merged 4 commits intomainfrom
lvkv/resolv-dep

Conversation

@lvkv
Copy link
Copy Markdown
Member

@lvkv lvkv commented Apr 16, 2026

The 0.4 versions of resolv and libresolv are on crates.io now (see mikedilger/resolv-rs#11). This updates Cargo.toml accordingly.

@lvkv lvkv requested a review from Copilot April 16, 2026 15:28
@lvkv lvkv self-assigned this Apr 16, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the resolv dependency configuration to use the crates.io release instead of a Git dependency, now that resolv v0.4 is published.

Changes:

  • Replaces the resolv Git dependency (pinned rev) with a crates.io versioned dependency (version = "0.4").
  • Removes now-obsolete explanatory comments related to the unreleased state of resolv v0.4.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Cargo.toml
@lvkv lvkv force-pushed the lvkv/resolv-dep branch from d2d5e51 to a31d491 Compare April 16, 2026 15:42
@lvkv lvkv force-pushed the lvkv/resolv-dep branch from a31d491 to 92c7d8e Compare April 16, 2026 15:43
@lvkv lvkv merged commit 93ab5a3 into main Apr 16, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants