Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

examples: fix up #64

Closed
wants to merge 4 commits into from
Closed

examples: fix up #64

wants to merge 4 commits into from

Commits on May 13, 2024

  1. test: refactor desugar test

    It should test the directive directly instead of going through the
    omnifset parsing as well.
    
    Signed-off-by: Simon de Vlieger <[email protected]>
    supakeen committed May 13, 2024
    Configuration menu
    Copy the full SHA
    a5d8205 View commit details
    Browse the repository at this point in the history
  2. test: expand desugar test

    Desugar is currently not correctly replacing multiple variables in a
    string. This adds failing test cases for that.
    
    Signed-off-by: Simon de Vlieger <[email protected]>
    supakeen committed May 13, 2024
    Configuration menu
    Copy the full SHA
    2af5ffc View commit details
    Browse the repository at this point in the history
  3. directive: make desugar do multiple substrings

    Use regular expressions to do replacements. Previously we didn't replace
    multiple occurences of variables correctly.
    
    Signed-off-by: Simon de Vlieger <[email protected]>
    supakeen committed May 13, 2024
    Configuration menu
    Copy the full SHA
    312f3dd View commit details
    Browse the repository at this point in the history
  4. make omnifests work on main

    Sadly a lot of the deduplication work that has been done has to be
    undone for now due to the way directives work. Let's get main working
    then put them back in in feature branches.
    
    Signed-off-by: Simon de Vlieger <[email protected]>
    supakeen committed May 13, 2024
    Configuration menu
    Copy the full SHA
    14065bc View commit details
    Browse the repository at this point in the history