-
Notifications
You must be signed in to change notification settings - Fork 1
Upgrade baton-sdk #103
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
Upgrade baton-sdk #103
Conversation
WalkthroughUpdated dependency versions in go.mod, including baton-sdk (v0.6.5 to v0.6.7), purego, gopsutil, tklauser modules, and golang.org/x/sys. No functional or API changes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
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
🧹 Nitpick comments (1)
go.mod (1)
7-7: Verify dependency versions and test compatibility.This PR updates multiple dependencies including baton-sdk from v0.6.5 to v0.6.7, along with indirect dependency updates (purego, gopsutil, and tklauser modules). Before merging:
- Confirm all updated versions exist and are stable releases
- Check for breaking changes in baton-sdk v0.6.6 and v0.6.7
- Verify the code builds and tests pass with these new versions
- Confirm no known security vulnerabilities in the updated packages
Run
go mod tidy,go build ./..., andgo test ./...to validate the updates resolve correctly and maintain compatibility.Also applies to: 62-62, 97-97, 105-105, 106-106, 128-128
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (135)
go.sumis excluded by!**/*.sumand included by nonevendor/github.com/conductorone/baton-sdk/internal/connector/connector.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/internal/connector/connector_server_unix.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pb/c1/connector/v2/annotation_security_insight.pb.gois excluded by!**/*.pb.go,!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pb/c1/connector/v2/annotation_security_insight.pb.validate.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pb/c1/connector/v2/annotation_security_insight_protoopaque.pb.gois excluded by!**/*.pb.go,!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pb/c1/connector/v2/resource.pb.gois excluded by!**/*.pb.go,!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pb/c1/connector/v2/resource.pb.validate.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pb/c1/connector/v2/resource_protoopaque.pb.gois excluded by!**/*.pb.go,!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pb/c1/connectorapi/baton/v1/baton.pb.gois excluded by!**/*.pb.go,!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pb/c1/connectorapi/baton/v1/baton.pb.validate.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pb/c1/connectorapi/baton/v1/baton_protoopaque.pb.gois excluded by!**/*.pb.go,!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pb/c1/reader/v2/grant.pb.gois excluded by!**/*.pb.go,!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pb/c1/reader/v2/grant_protoopaque.pb.gois excluded by!**/*.pb.go,!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/cli/commands.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/connectorbuilder/accounts.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/connectorrunner/runner.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/dotc1z/assets.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/dotc1z/c1file.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/dotc1z/decoder.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/dotc1z/diff.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/dotc1z/entitlements.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/dotc1z/file.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/dotc1z/grants.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/dotc1z/resouce_types.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/dotc1z/resources.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/dotc1z/session_store.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/dotc1z/sql_helpers.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/dotc1z/sync_runs.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/sdk/version.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/session/json_session.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/session/session.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/sync/expand/expander.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/sync/expand/graph.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/sync/expand/scc/bitset.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/sync/syncer.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/synccompactor/compactor.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/tasks/c1api/full_sync.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/tasks/c1api/hello.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/tasks/c1api/manager.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/tasks/local/action_invoker.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/tasks/local/syncer.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/types/resource/security_insight_trait.gois excluded by!vendor/**and included by nonevendor/github.com/conductorone/baton-sdk/pkg/types/sessions/sessions.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/README.mdis excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/abi_loong64.his excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/cgo.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/dlerror.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/dlfcn.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/dlfcn_netbsd.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/dlfcn_nocgo_netbsd.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/dlfcn_stubs.sis excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/func.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/gen.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/go_runtime.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/cgo/dlfcn_cgo_unix.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/cgo/syscall_cgo_unix.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/abi_loong64.his excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/asm_loong64.sis excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/callbacks.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/doc.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/go_libinit.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/go_linux_loong64.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/go_netbsd.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/go_setenv.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/go_util.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/iscgo.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/libcgo.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/libcgo_darwin.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/libcgo_freebsd.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/libcgo_linux.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/libcgo_netbsd.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/netbsd.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/setenv.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/symbols.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/symbols_darwin.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/symbols_freebsd.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/symbols_linux.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/symbols_netbsd.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/trampolines_loong64.sis excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/internal/fakecgo/trampolines_stubs.sis excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/nocgo.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/struct_amd64.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/struct_arm64.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/struct_loong64.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/struct_other.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/sys_amd64.sis excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/sys_arm64.sis excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/sys_loong64.sis excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/sys_unix_arm64.sis excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/sys_unix_loong64.sis excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/syscall.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/syscall_cgo_linux.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/syscall_sysv.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/syscall_windows.gois excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/zcallback_amd64.sis excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/zcallback_arm64.sis excluded by!vendor/**and included by nonevendor/github.com/ebitengine/purego/zcallback_loong64.sis excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/cpu/cpu_windows.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/host/host_aix_ppc64.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/internal/common/common.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/internal/common/common_darwin.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/internal/common/common_freebsd.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/internal/common/common_linux.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/internal/common/common_netbsd.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/internal/common/common_openbsd.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/internal/common/common_windows.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/internal/common/warnings.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/net/net_linux.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/net/net_windows.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/process/process.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/process/process_darwin.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/process/process_linux.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/process/process_solaris.gois excluded by!vendor/**and included by nonevendor/github.com/shirou/gopsutil/v4/process/process_windows.gois excluded by!vendor/**and included by nonevendor/github.com/tklauser/go-sysconf/.cirrus.ymlis excluded by!vendor/**and included by nonevendor/github.com/tklauser/go-sysconf/sysconf_netbsd.gois excluded by!vendor/**and included by nonevendor/github.com/tklauser/numcpus/.cirrus.ymlis excluded by!vendor/**and included by nonevendor/github.com/tklauser/numcpus/numcpus_linux.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/cpu/cpu.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/cpu/cpu_arm64.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/cpu/cpu_arm64.sis excluded by!vendor/**and included by nonevendor/golang.org/x/sys/cpu/cpu_gc_arm64.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/cpu/cpu_gccgo_arm64.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/cpu/cpu_netbsd_arm64.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/cpu/cpu_openbsd_arm64.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/unix/mkerrors.shis excluded by!vendor/**and included by nonevendor/golang.org/x/sys/unix/syscall_linux.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/unix/zerrors_linux.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/unix/zsyscall_linux.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/unix/ztypes_linux.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/windows/syscall_windows.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/windows/types_windows.gois excluded by!vendor/**and included by nonevendor/golang.org/x/sys/windows/zsyscall_windows.gois excluded by!vendor/**and included by nonevendor/modules.txtis excluded by!vendor/**and included by none
📒 Files selected for processing (1)
go.mod(4 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: go-lint
- GitHub Check: go-test (ubuntu-latest)
Description
Useful links:
Summary by CodeRabbit
Release Notes
✏️ Tip: You can customize this high-level summary in your review settings.