-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
build(deps): bump github.com/eclipse/paho.mqtt.golang from 1.4.3 to 1.5.0 #267
build(deps): bump github.com/eclipse/paho.mqtt.golang from 1.4.3 to 1.5.0 #267
Conversation
Caution Review failedThe pull request is closed. WalkthroughThe Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
go.sum
is excluded by!**/*.sum
Files selected for processing (1)
- go.mod (4 hunks)
Additional comments not posted (4)
go.mod (4)
24-24
: LGTM!The update to
golang.org/x/net v0.27.0
is approved.The code changes are approved.
39-39
: LGTM!The update to
github.com/gorilla/websocket v1.5.3
is approved.The code changes are approved.
78-78
: LGTM!The addition of
gopkg.in/yaml.v3 v3.0.1
as an indirect dependency is approved.The code changes are approved.
8-8
: Verify Go version requirement.The update to
github.com/eclipse/paho.mqtt.golang v1.5.0
requires Go version 1.20 or higher. Ensure that the project is using Go 1.20 or higher.Run the following script to verify the Go version used in the project:
Verification successful
Go version requirement is satisfied.
The
go.mod
file specifies that the project is using Go version 1.22.3, which meets the requirement forgithub.com/eclipse/paho.mqtt.golang v1.5.0
that requires Go 1.20 or higher.
go.mod
:go 1.22.3
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the Go version used in the project. # Test: Check the Go version. Expect: Go version 1.20 or higher. go versionLength of output: 82
Script:
#!/bin/bash # Search for the `go` directive in `go.mod` to check if the Go version is specified. rg '^go [0-9]+\.[0-9]+' go.mod # Search for any mention of Go version in documentation files like README.md. rg 'Go [0-9]+\.[0-9]+' README.mdLength of output: 77
Bumps [github.com/eclipse/paho.mqtt.golang](https://github.com/eclipse/paho.mqtt.golang) from 1.4.3 to 1.5.0. - [Release notes](https://github.com/eclipse/paho.mqtt.golang/releases) - [Commits](eclipse-paho/paho.mqtt.golang@v1.4.3...v1.5.0) --- updated-dependencies: - dependency-name: github.com/eclipse/paho.mqtt.golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
78d3f8a
to
c5a0691
Compare
Rebasing might not happen immediately, so don't worry if this takes some time.
Note: if you make any changes to this PR yourself, they will take precedence over the rebase.
Bumps github.com/eclipse/paho.mqtt.golang from 1.4.3 to 1.5.0.
Release notes
Sourced from github.com/eclipse/paho.mqtt.golang's releases.
Commits
714f7c0
Merge pull request #683 from ChIoT-Tech/master8768f3b
Update dependencies6801721
Resolve potential goroutine leak wien Disconnect called71f9814
fix: issue 675:goroutine leak when connectionUp(true) return error386b731
fix: issue 675:goroutine leak when connectionUp(true) return errorfe38f80
Resolve "cannot use os.ReadDir" introduced in PR #665b7215e4
Merge pull request #665 from vruge/patch-11a63b63
Merge pull request #671 from DVasselli/clientTimer8b638fb
Remove the channel draining since it is not reused.d0120ee
Replace the time.After with the timer for efficiency.Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditions
will show all of the ignore conditions of the specified dependency@dependabot ignore this major version
will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor version
will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependency
will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)