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

Allow regex pattern for expected student files. #387

Open
james-perretta opened this issue Nov 18, 2018 · 2 comments
Open

Allow regex pattern for expected student files. #387

james-perretta opened this issue Nov 18, 2018 · 2 comments
Labels
feature Adds some new functionality priority-1-normal requires custom migration Requires custom migration code to shift data into a new format size-medium

Comments

@james-perretta
Copy link
Contributor

james-perretta commented Nov 18, 2018

This would be in addition to shell-style wildcards.

@james-perretta
Copy link
Contributor Author

james-perretta commented Mar 6, 2019

Proposed design:

  • Enum with 3 options: exact match, shell wildcard, or regex.
  • When exact match, use ==
  • When shell wildcard, use fnmatch
  • When regex, use re.match

Note that this will need a custom migration in order to add the new enum field. We can use the same auto-detect logic from the frontend code to populate it.

@james-perretta james-perretta changed the title Allow regex instead of shell-style wildcard pattern for expected student files. Allow regex pattern for expected student files. Jul 7, 2020
@james-perretta
Copy link
Contributor Author

There hasn't been much demand for this. Closing unless it gets requested more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adds some new functionality priority-1-normal requires custom migration Requires custom migration code to shift data into a new format size-medium
Projects
None yet
Development

No branches or pull requests

1 participant