Skip to content
Snippets Groups Projects

make graphRegex simpler

Merged vincent lorrain requested to merge devfix into dev
All threads resolved!

Context

In issue Bug : Advanced node testing with Graph Regex an... (#86 - closed), the problem arises from the syntax of the lambda declaration, which is not intuitive.This modification aims to automatically add the lambda key as a test and eliminate the need to do it in two steps. It also adds the ability to accept True and False as bool in expressions.

Modified files

Add the automatically add the lambda key in :

src/graphRegex/GraphRegex.cpp

Add the True and False :

src/nodeTester/ConditionalLexer.cpp

CI :

unit_tests/graphRegex/Test_GraphRegex.cpp

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Maxence Naud resolved all threads

    resolved all threads

  • Maxence Naud approved this merge request

    approved this merge request

  • changed milestone to %aidge_core - v0.2.0

  • Maxence Naud enabled an automatic merge when the pipeline for 8a1edc15 succeeds

    enabled an automatic merge when the pipeline for 8a1edc15 succeeds

  • Maxence Naud mentioned in commit 93500677

    mentioned in commit 93500677

  • merged

  • mentioned in issue #86 (closed)

  • Please register or sign in to reply
    Loading