Skip to content

Conversation

@mtovino-cloudflare
Copy link
Contributor

  • the .build() call is required to align with recent rework of serialization
  • 'Type' was unused

I, [email protected], am both a rust n00b and a wirefilter n00b. I tried building the example so that I could tinker and learn, and needed to make these changes for the example to build and lint.

@mtovino-cloudflare mtovino-cloudflare force-pushed the mtovino/update_readme branch 2 times, most recently from 93df367 to de3f7e9 Compare August 8, 2025 21:39
- the .build() call is required to align with recent rework of
  serialization
- 'Type' was unused

The example is intended to match the one in
'engine/src/lib.rs' - which has also been updated to remove
the unused 'Type' and also 'rustfmt' the .build() call.
@zakcutner zakcutner merged commit 72e3954 into cloudflare:master Aug 8, 2025
7 checks passed
@mtovino-cloudflare mtovino-cloudflare deleted the mtovino/update_readme branch August 8, 2025 23:14
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