Skip to content

v1.18: support Pubkey::to_bytes as constexpr (backport of #220)#1107

Merged
joncinque merged 1 commit intov1.18from
mergify/bp/v1.18/pr-220
May 4, 2024
Merged

v1.18: support Pubkey::to_bytes as constexpr (backport of #220)#1107
joncinque merged 1 commit intov1.18from
mergify/bp/v1.18/pr-220

Conversation

@mergify
Copy link
Copy Markdown

@mergify mergify Bot commented Apr 29, 2024

Problem

Cannot do compile-time tricks with static const pubkeys

Summary of Changes

Adds support for constexprs

Fixes #


This is an automatic backport of pull request #220 done by [Mergify](https://mergify.com).

@mergify mergify Bot requested a review from a team as a code owner April 29, 2024 19:28
@mergify mergify Bot requested a review from a team April 29, 2024 19:28
@joncinque joncinque requested a review from t-nelson April 29, 2024 20:28
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.6%. Comparing base (c8e6fd0) to head (a9d3c15).

Additional details and impacted files
@@           Coverage Diff           @@
##            v1.18    #1107   +/-   ##
=======================================
  Coverage    81.6%    81.6%           
=======================================
  Files         827      827           
  Lines      225352   225352           
=======================================
+ Hits       184011   184022   +11     
+ Misses      41341    41330   -11     

Copy link
Copy Markdown

@steviez steviez left a comment

Choose a reason for hiding this comment

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

Was a little surprised we marked this one for BP, but very small / low risk change so I'm ok with it

@joncinque joncinque merged commit a73b6cb into v1.18 May 4, 2024
@joncinque joncinque deleted the mergify/bp/v1.18/pr-220 branch May 4, 2024 13:33
anwayde pushed a commit to firedancer-io/agave that referenced this pull request Jul 23, 2024
… (anza-xyz#1107)

sdk: support Pubkey::to_bytes as constexpr (anza-xyz#220)

(cherry picked from commit 9b16df2)

Co-authored-by: ripatel-fd <ripatel+git@jumptrading.com>
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.

6 participants