Skip to content

feat: Add pbj.useUtils to allow projects to customize their codec.parse bytes impl - #939

Open
ldintr wants to merge 1 commit into
ldintr-zzfrom
ldintr-pbjUseUtil
Open

feat: Add pbj.useUtils to allow projects to customize their codec.parse bytes impl#939
ldintr wants to merge 1 commit into
ldintr-zzfrom
ldintr-pbjUseUtil

Conversation

@ldintr

@ldintr ldintr commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Description:
The codec generator will create calls to parse(Bytes) and some projects may want to use thread-local storage. This commit adds an option that will generate PbjUtils.parse instead so those projects can use thread-local storage

Related issue(s):

Fixes #

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@ldintr
ldintr added this pull request to stack #908 September 9, 2026 21:06
@ldintr
ldintr requested review from a team as code owners September 9, 2026 21:06
@ldintr ldintr changed the title Ldintr pbj use util feat: Add pbj.useUtils to allow projects to customize their codec.parse bytes impl Sep 9, 2026
@ldintr
ldintr force-pushed the ldintr-pbjUseUtil branch 2 times, most recently from 298e3dd to e44b630 Compare September 9, 2026 21:17
@ldintr ldintr self-assigned this Sep 9, 2026
…es impl

Signed-off-by: ldintr <levo.d@swirldslabs.com>
@ldintr
ldintr requested review from a team as code owners September 10, 2026 19:06
@github-actions

Copy link
Copy Markdown

JUnit Test Report

   523 files  ±0     523 suites  ±0   30s ⏱️ -1s
 1 645 tests +3   1 641 ✅ +3   4 💤 ±0  0 ❌ ±0 
10 863 runs  +3  10 835 ✅ +3  28 💤 ±0  0 ❌ ±0 

Results for commit f3aa552. ± Comparison against base commit 4bc3c7a.

@github-actions

Copy link
Copy Markdown

Integration Test Report

    428 files  ±0      428 suites  ±0   26m 42s ⏱️ + 3m 24s
115 048 tests ±0  115 048 ✅ ±0  0 💤 ±0  0 ❌ ±0 
115 292 runs  ±0  115 292 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit f3aa552. ± Comparison against base commit 4bc3c7a.

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.

1 participant