Skip to content

Conversation

CardboardTurkey
Copy link
Collaborator

@CardboardTurkey CardboardTurkey commented Jul 4, 2025

  • test: add test for known bug in comparison
    Signed-off-by: Kiran Ostrolenk kiran@ostrolenk.co.uk

  • docs(README): fix markdown issues
    See markdownlint for more info.

    Signed-off-by: Kiran Ostrolenk kiran@ostrolenk.co.uk

  • chore: cargo fmt
    Signed-off-by: Kiran Ostrolenk kiran@ostrolenk.co.uk

  • feat: switch to using the builder pattern for improved type safety
    Configuring the parsing of patterns is now managed using the builder
    pattern.

    Signed-off-by: Kiran Ostrolenk kiran@ostrolenk.co.uk

Closes #19

@CardboardTurkey
Copy link
Collaborator Author

Ah merge conflicts

Signed-off-by: Kiran Ostrolenk <kiran@ostrolenk.co.uk>
See [markdownlint] for more info.

[markdownlint]: https://github.com/DavidAnson/markdownlint/blob/main/doc/Rules.md

Signed-off-by: Kiran Ostrolenk <kiran@ostrolenk.co.uk>
Signed-off-by: Kiran Ostrolenk <kiran@ostrolenk.co.uk>
Configuring the parsing of patterns is now managed using the builder
pattern.

Signed-off-by: Kiran Ostrolenk <kiran@ostrolenk.co.uk>
@CardboardTurkey
Copy link
Collaborator Author

I moved some of the tests from pattern.rs to parser.rs because they felt more related to parsing than pattern handling. git diff --color-moved might help with reviewing that.

@Hexagon Hexagon merged commit 9744a55 into Hexagon:main Jul 5, 2025
1 check passed
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.

Parse, don't validate?

2 participants