Skip to content

Update go deps (major) - #834

Open
renovate[bot] wants to merge 1 commit into
mainfrom
renovate/major-go-deps
Open

Update go deps (major)#834
renovate[bot] wants to merge 1 commit into
mainfrom
renovate/major-go-deps

Conversation

@renovate

@renovate renovate Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Confidence
github.com/mjibson/go-dsp v0.0.0-20180508042940-11479a337f12v1.0.0 age confidence
github.com/ory/dockertest/v3 v3.12.0v4.0.0 age confidence
github.com/pion/rtp v1.10.5v2.0.0 age confidence

Release Notes

ory/dockertest (github.com/ory/dockertest/v3)

v4.0.0

Compare Source

What's Changed

New Contributors

Full Changelog: ory/dockertest@v3.12.0...v4.0.0

pion/rtp (github.com/pion/rtp)

v2.0.0

Compare Source


Configuration

📅 Schedule: (UTC)

  • Branch creation
    • "before 9am on monday"
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate
renovate Bot requested a review from a team as a code owner September 7, 2026 07:58

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Devin Review

Comment thread go.mod
Comment on lines +19 to +20
github.com/ory/dockertest/v4 v4.0.0
github.com/pion/rtp/v2 v2.0.0

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Major upgrades never reach consumers

Declaring dockertest/v4 and rtp/v2 leaves every consumer on the old module paths. RTP handling remains on version 1, while integration tests still require dockertest/v3.

Prompt for agents
Complete both major-version migrations. Update all github.com/pion/rtp imports in pkg/media/rtpconn/conn.go, pkg/sip/media.go, and pkg/sip/media_pipeline_test.go to github.com/pion/rtp/v2, adapting API changes as needed. Update all github.com/ory/dockertest/v3 imports under test/integration to github.com/ory/dockertest/v4, including the docker subpackage, and adapt the v4 API. Then regenerate go.mod and go.sum so the old modules are removed and the new modules are actually selected.
Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Generated by renovateBot
@renovate
renovate Bot force-pushed the renovate/major-go-deps branch from a562921 to da8898a Compare September 7, 2026 19:09
@renovate

renovate Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor Author

⚠️ Artifact update problem

Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.

♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

  • any of the package files in this branch needs updating, or
  • the branch becomes conflicted, or
  • you click the rebase/retry checkbox if found above, or
  • you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:

File name: go.sum
Command failed: go get -t ./...
go: github.com/mjibson/go-dsp@v1.0.0: parsing go.mod:
	module declares its path as: github.com/madelynnblue/go-dsp
	        but was required as: github.com/mjibson/go-dsp

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.

0 participants