Skip to content

Conversation

@philpax
Copy link
Member

@philpax philpax commented Nov 18, 2025

Updated the wikitext_simplified dependency to the latest version which introduced the Spanned wrapper type for nodes to include source location information.

Updated the wikitext_simplified dependency to the latest version which
introduced the Spanned<T> wrapper type for nodes to include source
location information. Made the following changes:

- Updated Cargo.lock with new wikitext_simplified commit (e637ff1b)
- Added Spanned and Span imports to both main.rs and template.rs
- Created helper function spanned() to wrap nodes with default spans
- Updated pattern matching to access .value field on Spanned nodes
- Modified children vectors to use Vec<Spanned<WikitextSimplifiedNode>>
- Updated function signatures to handle Spanned types correctly
- Fixed all compilation errors related to the API changes

All tests pass successfully.
Created a new util module to hold common helper functions and moved the
spanned helper there, renaming it to empty_spanned for clarity.

Changes:
- Created src/util.rs with empty_spanned() function
- Updated main.rs to import and use empty_spanned from util
- Updated template.rs to import and use empty_spanned from util
- Removed duplicate spanned() definitions from both files
- Removed unused Spanned import from template.rs

This reduces code duplication and provides a central location for
common utility functions.
@philpax philpax marked this pull request as ready for review November 18, 2025 08:40
@philpax philpax merged commit 5e67bad into main Nov 18, 2025
3 checks passed
@philpax philpax deleted the claude/update-wikitext-01PP4XEfxrn9RHWZDNA3Xv8y branch November 18, 2025 08:40
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.

3 participants