New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

add validate_regexes.rs #21

Merged

swanandx merged 1 commit into swanandx:main from macroexpansion:main

Oct 11, 2022

Contributor

macroexpansion commented Oct 10, 2022

add test in validate_regexes.rs and fix some regexes to make the test work.

macroexpansion force-pushed the main branch from c24ab30 to e918d23 Compare

October 10, 2022 12:47

swanandx reviewed

View reviewed changes

tests/validate_regexes.rs Outdated Show resolved Hide resolved

swanandx reviewed

View reviewed changes

tests/validate_regexes.rs Outdated Show resolved Hide resolved

swanandx reviewed

View reviewed changes

tests/validate_regexes.rs Outdated Show resolved Hide resolved

swanandx reviewed

View reviewed changes

tests/validate_regexes.rs Outdated Show resolved Hide resolved

swanandx reviewed

View reviewed changes

Owner

swanandx left a comment •

edited

Loading

Please fix Clippy warnings and do mentioned changes.

Also rename the test from testcase_works to something which describes what its doing. e.g. validate_regex_examples

macroexpansion force-pushed the main branch from 3d8aa9c to ad16fca Compare

October 11, 2022 02:27

swanandx reviewed

View reviewed changes

tests/validate_regexes.rs Outdated Show resolved Hide resolved


          add validate_regexes.rs

c4b7c75

macroexpansion force-pushed the main branch from ad16fca to c4b7c75 Compare

October 11, 2022 12:19

Owner

swanandx commented Oct 11, 2022

Thank you for contributing!

swanandx merged commit df7825b into swanandx:main

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet