-
Notifications
You must be signed in to change notification settings - Fork 20
Open
Labels
Description
Currently on the main branch the SQL Injection session drills only contain a sad .sh script each 😢
There is a PR open #73 that nicely adds proper details for each drill, like description of the drill and of the solution, source code, deploy files.
The task is to review & update the PR until it arrives in a ready-to-merge form, in which case just ping one of the team members to take a final look on it:
- please check if it follows the Open Education Hub repository template structure; the PR could have been created when we were still on an outdated structure
- compare with the drills from the other sessions to have an idea of the final form
- try to locally deploy each drill and test if the provided solution script works (run the script)
- statically review the code, the description for typos, coding style etc.
Reactions are currently unavailable