| Commit message (Collapse) | Author | Age | Lines | |
|---|---|---|---|---|
| * | Revert "Add basic logger for responses" | 2019-06-05 | -16/+1 | |
| | | | | | | Gunicorn's access logger makes the middleware redundant. This reverts commit c1c854e0ad4deb5c4fa8eb2a35b43a042f15255c. | |||
| * | Add logging for NsJail | 2019-06-05 | -28/+90 | |
| | | | | | | | | | | | | NsJail's is configured to log to a temporary file rather than stderr. The contents of the file are parsed using regex after the process exits. When not debugging, some blacklisted messages and most info-level messages are skipped. * Add a snekbox logger * Log the Python code being executed if debugging * Use nested single quotes in a test to fix a linter error | |||
| * | Merge remote-tracking branch 'origin/revitalisation' into refactor/nsjail | 2019-06-05 | -136/+257 | |
| |\ | ||||
| | * | Merge pull request #23 from python-discord/falcon | 2019-06-02 | -163/+278 | |
| | |\ | | | | | | | Replace Flask with Falcon | |||
| | | * | Revise docstrings for SnekAPI & EvalResource | 2019-05-29 | -9/+9 | |
| | | | | ||||
| | | * | Add basic logger for responses | 2019-05-29 | -1/+16 | |
| | | | | ||||
| | | * | Document the API | 2019-05-29 | -0/+50 | |
| | | | | ||||
| | | * | Move SnekAPI import back to top of module | 2019-05-29 | -1/+2 | |
| | | | | ||||
| | | * | Enable branch coverage & disable app.py coverage | 2019-05-29 | -9/+5 | |
| | | | | ||||
| | | * | Add API tests for eval resource | 2019-05-29 | -1/+65 | |
| | | | | ||||
| | | * | Separate application instance from definition | 2019-05-29 | -6/+16 | |
| | | | | | | | | | | | | | | | | This way, it is easier for tests to use the application because they can instead create their own instances after patching. | |||
| | | * | Validate request's input | 2019-05-28 | -2/+46 | |
| | | | | ||||
| | | * | Create falcon API | 2019-05-28 | -59/+36 | |
| | | | | ||||
| | | * | Replace flask dependency with falcon | 2019-05-28 | -99/+57 | |
| | |/ | ||||
| * | | Configure gunicorn logging | 2019-06-04 | -10/+24 | |
| | | | ||||
| * | | Lint tests | 2019-05-30 | -6/+4 | |
| | | | | | | | | | | | * Ignore D1xx for tests because they shouldn't require docstrings * Update pre-commit-hooks repo to v2.2.3 | |||
| * | | Revert "Run flake8 via pre-commit for the pipenv lint script" | 2019-05-30 | -7/+1 | |
| | | | | | | | | | This reverts commit c2416eb672afc80476fe2da1b364af5d018a16b0. | |||
| * | | Revert "Lint tests" | 2019-05-30 | -30/+1 | |
| | | | | | | | | | This reverts commit e75c764f693c3688a59af0d679e0d3e94f003503. | |||
| * | | Refactor NsJail | 2019-05-30 | -67/+74 | |
| | | | ||||
| * | | Run flake8 via pre-commit for the pipenv lint script | 2019-05-30 | -1/+7 | |
| | | | | | | | | | | | | | | | | | | | Pre-commit is a nice wrapper for this particularily because it allows both the snekbox and tests to be linted with a single command while clearly distinguishing their results. The Python script is a cross-platform implementation of the following: git ls-files -- '*.py' | xargs pre-commit run flake8 --files | |||
| * | | Lint tests | 2019-05-30 | -1/+30 | |
| | | | | | | | | | | | Tests ignore all D1xx warnings because tests shouldn't require docstrings. | |||
| * | | Fix pre-commit configuration | 2019-05-30 | -1/+14 | |
| | | | | | | | | | | | | | | | | | | | | | | | Excluded files and directories in the flake8 config file are overwritten by the --files argument, which is considered intended behaviour. However, this is problematic due to pre-commit's reliance on the --files argument. Therefore, pre-commit's own exclude config option must be leveraged. * Properly exclude tests directory * Add flake8 plugins as dependencies so that they are actually used | |||
| * | | Add flake8 plugin to only allow double quotes | 2019-05-29 | -103/+119 | |
| |/ | ||||
| * | Merge branch 'master' into revitalisation | 2019-05-28 | -9/+58 | |
| |\ | ||||
| | * | Merge pull request #21 from python-discord/docstring-lint-change | 2019-04-29 | -9/+1 | |
| | |\ | | | | | | | Docstring lint change | |||
| | | * | Remove space between ignored flake8 rules | 2019-04-24 | -1/+1 | |
| | | | | ||||
| | | * | Put back D400 as ignored code | 2019-04-24 | -1/+1 | |
| | | | | | | | | | | | | | Oops | |||
| | | * | Relint Snekbox with new linting rules | 2019-04-24 | -8/+0 | |
| | | | | ||||
| | | * | Remove docstring newline linting from ignored codes | 2019-04-24 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | Remove docstring-ends-with-period enforcement to allow for freedom of punctuation. Removes D202, D204, D400 | |||
| | * | | Merge pull request #19 from python-discord/urllib3-vuln-fix | 2019-04-29 | -45/+39 | |
| | |\ \ | | |/ | |/| | Manually pin urllib3 version to mitigate vulnerability | |||
| | | * | Manually pin urllib3 version to mitigate vulnerability | 2019-04-21 | -45/+39 | |
| | |/ | ||||
| | * | Merge pull request #18 from python-discord/update-contrib-doc | 2019-04-04 | -4/+57 | |
| | |\ | | | | | | | | | | | | | Update contributor doc | |||
| | | * | Update contributor doc | 2019-04-03 | -4/+57 | |
| | |/ | ||||
| * | | Merge pull request #17 from python-discord/docker | 2019-05-28 | -65/+109 | |
| |\ \ | | | | | | | Restructure Docker Images | |||
| | * | | Revert "Always build images if they do not exist" | 2019-05-28 | -6/+2 | |
| | | | | | | | | | | | | | This reverts commit 02e348ac249c1aba2d378ecda9af2a9a8ac1c894. | |||
| | * | | Build venv image in PRs when Pipfiles change | 2019-05-28 | -13/+5 | |
| | | | | ||||
| | * | | Don't install dev dependencies in Docker image | 2019-05-13 | -1/+1 | |
| | | | | ||||
| | * | | Exlcude everything, then make exceptions in dockerignore | 2019-05-13 | -18/+8 | |
| | | | | ||||
| | * | | Never build the final image in a PR | 2019-05-13 | -0/+1 | |
| | | | | ||||
| | * | | Disable pip cache | 2019-05-13 | -2/+5 | |
| | | | | ||||
| | * | | Always build images if they do not exist | 2019-05-13 | -9/+15 | |
| | | | | ||||
| | * | | Build images if their Dockerfiles change even in PRs | 2019-05-13 | -6/+12 | |
| | | | | | | | | | | | | | The venv image will still not be built in a PR if a Pipfile changes. | |||
| | * | | Revert "Remove pushing images for testing CI" | 2019-05-13 | -3/+4 | |
| | | | | | | | | | | | | | This reverts commit c66b10f8820907fb7065cbede2e87c541da6029b. | |||
| | * | | Use alternate syntax for variables | 2019-05-13 | -4/+4 | |
| | | | | ||||
| | * | | Make the JSON regex more robust | 2019-05-12 | -2/+2 | |
| | | | | ||||
| | * | | Use eq() function in conditions instead of relying on casts | 2019-05-12 | -4/+4 | |
| | | | | ||||
| | * | | Fix missing fi for if statement | 2019-05-12 | -0/+1 | |
| | | | | ||||
| | * | | Retrieve commit from pr.sourceSha for PR builds | 2019-05-12 | -5/+9 | |
| | | | | ||||
| | * | | Revert "Ensure branches are prefixed with refs/heads/" | 2019-05-12 | -1/+0 | |
| | | | | | | | | | | | | | This reverts commit 27af05b05b5c5bc0974ed61bf75f93a3b42280df. | |||
| | * | | Reference target branch as a remote rather than a local branch | 2019-05-12 | -1/+1 | |
| | | | | ||||