Skip to content

inject CapDonID#2102

Draft
tarcisiozf wants to merge 2 commits into
mainfrom
CRE-4409-capability-don-id
Draft

inject CapDonID#2102
tarcisiozf wants to merge 2 commits into
mainfrom
CRE-4409-capability-don-id

Conversation

@tarcisiozf
Copy link
Copy Markdown

Requires

Supports

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 29, 2026

✅ API Diff Results - github.com/smartcontractkit/chainlink-common

✅ Compatible Changes (3)

pkg/capabilities/pb.(*InitialiseRequest) (1)
  • GetCapabilityDonId — ➕ Added
pkg/capabilities/pb.InitialiseRequest (1)
  • CapabilityDonId — ➕ Added
pkg/types/core.StandardCapabilitiesDependencies (1)
  • CapabilityDonID — ➕ Added

📄 View full apidiff report

@tarcisiozf tarcisiozf force-pushed the CRE-4409-capability-don-id branch from d972907 to 3f2cb4c Compare May 29, 2026 14:37
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 2.

Benchmark suite Current: 3480ca4 Previous: d972907 Ratio
BenchmarkKeystore_Sign/nop/out-of-process 90696 ns/op 26228 ns/op 3.46
BenchmarkKeystore_Sign/hex/out-of-process 91599 ns/op 26633 ns/op 3.44
BenchmarkKeystore_Sign/ed25519/out-of-process 139111 ns/op 46478 ns/op 2.99

This comment was automatically generated by workflow using github-action-benchmark.

Comment thread pkg/types/core/standard_capabilities_dependencies.go
@patrickhuie19 patrickhuie19 requested a review from jmank88 May 29, 2026 17:33
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