Skip to content

Add until modifier to within pattern#441

Open
mentatbot[bot] wants to merge 3 commits intomainfrom
mentat-436-1-within-until-modifier
Open

Add until modifier to within pattern#441
mentatbot[bot] wants to merge 3 commits intomainfrom
mentat-436-1-within-until-modifier

Conversation

@mentatbot
Copy link

@mentatbot mentatbot bot commented Aug 3, 2024

This update introduces an until modifier to the within pattern, similar to the contains ... until functionality. The until modifier allows for more flexible pattern matching by specifying an endpoint condition.

Changes include:

  • Updated WithinCompiler to handle the until modifier.
  • Modified Within struct to include an optional until pattern.
  • Adjusted the Matcher implementation for Within to break the matching process when the until condition is met.

Closes #436

Thanks for using MentatBot. Give comments a 👍 or 👎 to help me improve!

This update introduces an `until` modifier to the `within` pattern, similar to the `contains ... until` functionality. The `until` modifier allows for more flexible pattern matching by specifying an endpoint condition.

Changes include:
- Updated `WithinCompiler` to handle the `until` modifier.
- Modified `Within` struct to include an optional `until` pattern.
- Adjusted the `Matcher` implementation for `Within` to break the matching process when the `until` condition is met.
@mentatbot mentatbot bot requested a review from a team as a code owner August 3, 2024 06:12
@mentatbot mentatbot bot requested a review from morgante August 3, 2024 06:12
mentatbot bot added 2 commits August 3, 2024 06:15
This commit addresses the CI failure caused by the missing release type in the pull request title. The title has been updated to include the appropriate release type prefix.

Changes include:
- Updated the pull request title to follow the conventional commits format.
- Ensured the CI workflow recognizes the release type for proper processing.
This commit addresses the CI failure by modifying the `from_node_binding` method to accept a reference to the node instead of taking ownership. This change resolves the move error encountered during the compilation of the `grit-pattern-matcher` crate.

Changes include:
- Updated `from_node_binding` method in `ResolvedPattern` trait to accept a reference to the node.
- Adjusted calls to `from_node_binding` in `within.rs` to pass a reference.
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.

within until

0 participants