Skip to content

Solana: Add GetFiltersNames and proto helpers#1801

Merged
ilija42 merged 1 commit intomainfrom
solana-logtrigger-cap-bump
Feb 25, 2026
Merged

Solana: Add GetFiltersNames and proto helpers#1801
ilija42 merged 1 commit intomainfrom
solana-logtrigger-cap-bump

Conversation

@silaslenihan
Copy link
Contributor

@silaslenihan silaslenihan commented Jan 28, 2026 •
edited
Loading

@silaslenihan silaslenihan requested review from a team as code owners January 28, 2026 19:36
@github-actions
Copy link

github-actions bot commented Jan 28, 2026

👋 silaslenihan, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@github-actions
Copy link

github-actions bot commented Jan 28, 2026 •
edited
Loading

⚠️ API Diff Results - Breaking changes detected

📦 Module: github-com-smartcontractkit-chainlink-common

🔴 Breaking Changes (2)

pkg/chains/solana.SolanaClient (1)
  • GetFiltersNames — ➕ Added
pkg/types.SolanaService (1)
  • GetFiltersNames — ➕ Added

📄 View full apidiff report

@silaslenihan silaslenihan marked this pull request as draft January 28, 2026 19:39
@silaslenihan silaslenihan force-pushed the solana-logtrigger-cap-bump branch from 088974e to 465d52f Compare January 29, 2026 16:31
@silaslenihan silaslenihan force-pushed the solana-logtrigger-cap-bump branch from 0c02e62 to 5100fdf Compare January 29, 2026 20:02
github-actions[bot]
github-actions bot reviewed Jan 29, 2026
View reviewed changes
Copy link

@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: 5100fdf Previous: 088974e Ratio
BenchmarkKeystore_Sign/nop/in-process 784.5 ns/op 368.2 ns/op 2.13

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

@silaslenihan silaslenihan marked this pull request as ready for review February 5, 2026 19:06
dhaidashenko
dhaidashenko reviewed Feb 6, 2026
View reviewed changes
@silaslenihan silaslenihan force-pushed the solana-logtrigger-cap-bump branch from 87a4969 to 6b391ae Compare February 25, 2026 14:18
@ilija42 ilija42 added this pull request to the merge queue Feb 25, 2026
Merged via the queue into main with commit ca7f453 Feb 25, 2026
34 of 36 checks passed
@ilija42 ilija42 deleted the solana-logtrigger-cap-bump branch February 25, 2026 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

@yashnevatia yashnevatia yashnevatia approved these changes

@Unheilbar Unheilbar Unheilbar approved these changes

@dhaidashenko dhaidashenko dhaidashenko approved these changes

@ilija42 ilija42 ilija42 approved these changes

@Farber98 Farber98 Awaiting requested review from Farber98

@jmank88 jmank88 Awaiting requested review from jmank88

@michael-cll michael-cll Awaiting requested review from michael-cll

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@silaslenihan @yashnevatia @Unheilbar @dhaidashenko @ilija42