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

Feature Request: Add support for python regex in student test wildcards #36

Open
nelsontm opened this issue Mar 26, 2022 · 3 comments
Open

Comments

@nelsontm
Copy link

The current python fnmatch does not support disallowing certain characters from student file names, like spaces or '&'. This is causing errors for our makefile

@nelsontm nelsontm changed the title Add support for python regex in student test wildcards Feature Request: Add support for python regex in student test wildcards Mar 26, 2022
@james-perretta
Copy link
Contributor

Hmm I think fnmatch is supposed to support character exclusion with [!seq]. Have you tried that/did it work? If it doesn't work, then my first priority would be fixing that. Otherwise if there's a large enough demand for regexes in general I can consider reopening eecs-autograder/autograder-server#387

@nelsontm
Copy link
Author

That works for one character that is not a space. I'm trying to get a string of alphanumeric characters and from my understanding, fnmatch does not support repetition patterns like * or + from regex.

@james-perretta
Copy link
Contributor

Ah right, I was just mixing things up then. I'll go ahead and re-open the server-repo issue for this.

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

No branches or pull requests

2 participants