-
Notifications
You must be signed in to change notification settings - Fork 0
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
Bump go.uber.org/zap from 1.21.0 to 1.27.0 #6
base: master
Are you sure you want to change the base?
Conversation
Bumps [go.uber.org/zap](https://github.com/uber-go/zap) from 1.21.0 to 1.27.0. - [Release notes](https://github.com/uber-go/zap/releases) - [Changelog](https://github.com/uber-go/zap/blob/master/CHANGELOG.md) - [Commits](uber-go/zap@v1.21.0...v1.27.0) --- updated-dependencies: - dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Reviewer's Guide by SourceryThis pull request bumps the go.uber.org/zap dependency from version 1.21.0 to 1.27.0. This includes several new features and performance improvements. Class diagram for updated Zap Logger interfacesclassDiagram
class Logger {
+WithLazy() Logger
+Name() string
}
class SugaredLogger {
+WithLazy() SugaredLogger
+Log(level, args...)
+Logw(level, msg, keysAndValues...)
+Logln(level, args...)
}
class Field {
+Dict() Field
}
class TestingWriter {
+NewTestingWriter()
}
Logger -- SugaredLogger
Logger -- Field
note for Logger "Added WithLazy() for lazy context evaluation"
note for SugaredLogger "Added new logging methods with level parameter"
note for Field "Added Dict as a new field type"
note for TestingWriter "New public testing writer with more flexibility"
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the 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.
We have skipped reviewing this pull request. It seems to have been created by a bot (hey, dependabot[bot]!). We assume it knows what it's doing!
CI Feedback 🧐(Feedback updated until commit 2908534)A test triggered by this PR failed. Here is an AI-generated analysis of the failure:
|
Bumps go.uber.org/zap from 1.21.0 to 1.27.0.
Release notes
Sourced from go.uber.org/zap's releases.
... (truncated)
Changelog
Sourced from go.uber.org/zap's changelog.
... (truncated)
Commits
fcf8ee5
Release v1.27.0 (#1419)e5a56ee
Add WithPanicHook logger option for panic log tests (#1416)0e2aa4e
assets: Fix logo color profile (#1418)956a21c
Add methods for logging with level as argument (#1406)2a893f6
build(deps): bump golangci/golangci-lint-action from 3 to 4 (#1417)e5745d6
ci: Test with Go 1.22 (#1409)7db06bc
zapslog: rename Option to HandlerOption (#1411)35ded09
zapslog: fix all with slogtest, support inline group, ignore empty group. (#1...27b96e3
Make zaptest.NewTestingWriter public (#1399)70f61bb
zapslog: Bump zap from v1.24.0 to v1.26.0 (#1404)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)Summary by Sourcery
Update go.uber.org/zap from 1.21.0 to 1.27.0.
Enhancements:
Dict
as aField
.WithLazy
method.zap/exp/zapslog
for slog integration.Name
method to retrieve logger name.Str
andStrs
for String-like fields inzap/exp/expfield
.Any
.WithLazy
method forSugaredLogger
.NewTestingWriter
for more flexible testing.Log
,Logw
, andLogln
methods toSugaredLogger
.WithPanicHook
option.Tests: