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.

Sign up for GitHub

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

Jump to bottom

Fix: const constructors for PatternEvaluators #459

Open

simon-the-shark wants to merge 1 commit into joanpablo:master from simon-the-shark:master

simon-the-shark commented Jun 21, 2024

I've added const keyword in two places.

This change is needed to allow us to use PatternEvaluators with code generation version of this plugin like this:

@RfControl(validators: [
      RequiredValidator(),
      PatternValidator(
        DefaultPatternEvaluator(
          r'^(https:\/\/)[a-zA-Z0-9\-\.]+\.[a-z]{2,3}(/\S*)?$',
        ),
      )
    ])
    String? url,


          fix: const constructors for PatternEvaluators

46a732f

Owner

joanpablo commented Sep 18, 2024

Thanks @simon-the-shark.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet