aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | * Revert "Add basic logger for responses"Gravatar MarkKoz2019-06-05-16/+1
| | | | | | | | | | | | | | | Gunicorn's access logger makes the middleware redundant. This reverts commit c1c854e0ad4deb5c4fa8eb2a35b43a042f15255c.
| | * Add logging for NsJailGravatar MarkKoz2019-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/nsjailGravatar MarkKoz2019-06-05-136/+257
| | |\ | | |/ | |/|
| * | Merge pull request #23 from python-discord/falconGravatar Leon Sandøy2019-06-02-163/+278
| |\ \ | | | | | | | | Replace Flask with Falcon
| | * | Revise docstrings for SnekAPI & EvalResourceGravatar MarkKoz2019-05-29-9/+9
| | | |
| | * | Add basic logger for responsesGravatar MarkKoz2019-05-29-1/+16
| | | |
| | * | Document the APIGravatar MarkKoz2019-05-29-0/+50
| | | |
| | * | Move SnekAPI import back to top of moduleGravatar MarkKoz2019-05-29-1/+2
| | | |
| | * | Enable branch coverage & disable app.py coverageGravatar MarkKoz2019-05-29-9/+5
| | | |
| | * | Add API tests for eval resourceGravatar MarkKoz2019-05-29-1/+65
| | | |
| | * | Separate application instance from definitionGravatar MarkKoz2019-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 inputGravatar MarkKoz2019-05-28-2/+46
| | | |
| | * | Create falcon APIGravatar MarkKoz2019-05-28-59/+36
| | | |
| | * | Replace flask dependency with falconGravatar MarkKoz2019-05-28-99/+57
| |/ /
| | * Configure gunicorn loggingGravatar MarkKoz2019-06-04-10/+24
| | |
| | * Lint testsGravatar MarkKoz2019-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"Gravatar MarkKoz2019-05-30-7/+1
| | | | | | | | | | | | This reverts commit c2416eb672afc80476fe2da1b364af5d018a16b0.
| | * Revert "Lint tests"Gravatar MarkKoz2019-05-30-30/+1
| | | | | | | | | | | | This reverts commit e75c764f693c3688a59af0d679e0d3e94f003503.
| | * Refactor NsJailGravatar MarkKoz2019-05-30-67/+74
| | |
| | * Run flake8 via pre-commit for the pipenv lint scriptGravatar MarkKoz2019-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 testsGravatar MarkKoz2019-05-30-1/+30
| | | | | | | | | | | | | | | Tests ignore all D1xx warnings because tests shouldn't require docstrings.
| | * Fix pre-commit configurationGravatar MarkKoz2019-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 quotesGravatar MarkKoz2019-05-29-103/+119
| |/
| * Merge branch 'master' into revitalisationGravatar MarkKoz2019-05-28-9/+58
| |\
| * \ Merge pull request #17 from python-discord/dockerGravatar Mark2019-05-28-65/+109
| |\ \ | | | | | | | | Restructure Docker Images
| | * | Revert "Always build images if they do not exist"Gravatar MarkKoz2019-05-28-6/+2
| | | | | | | | | | | | | | | | This reverts commit 02e348ac249c1aba2d378ecda9af2a9a8ac1c894.
| | * | Build venv image in PRs when Pipfiles changeGravatar MarkKoz2019-05-28-13/+5
| | | |
| | * | Don't install dev dependencies in Docker imageGravatar MarkKoz2019-05-13-1/+1
| | | |
| | * | Exlcude everything, then make exceptions in dockerignoreGravatar MarkKoz2019-05-13-18/+8
| | | |
| | * | Never build the final image in a PRGravatar MarkKoz2019-05-13-0/+1
| | | |
| | * | Disable pip cacheGravatar MarkKoz2019-05-13-2/+5
| | | |
| | * | Always build images if they do not existGravatar MarkKoz2019-05-13-9/+15
| | | |
| | * | Build images if their Dockerfiles change even in PRsGravatar MarkKoz2019-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"Gravatar MarkKoz2019-05-13-3/+4
| | | | | | | | | | | | | | | | This reverts commit c66b10f8820907fb7065cbede2e87c541da6029b.
| | * | Use alternate syntax for variablesGravatar MarkKoz2019-05-13-4/+4
| | | |
| | * | Make the JSON regex more robustGravatar MarkKoz2019-05-12-2/+2
| | | |
| | * | Use eq() function in conditions instead of relying on castsGravatar MarkKoz2019-05-12-4/+4
| | | |
| | * | Fix missing fi for if statementGravatar MarkKoz2019-05-12-0/+1
| | | |
| | * | Retrieve commit from pr.sourceSha for PR buildsGravatar MarkKoz2019-05-12-5/+9
| | | |
| | * | Revert "Ensure branches are prefixed with refs/heads/"Gravatar MarkKoz2019-05-12-1/+0
| | | | | | | | | | | | | | | | This reverts commit 27af05b05b5c5bc0974ed61bf75f93a3b42280df.
| | * | Reference target branch as a remote rather than a local branchGravatar MarkKoz2019-05-12-1/+1
| | | |
| | * | Use true/false strings for settings booleans in bashGravatar MarkKoz2019-05-12-4/+4
| | | |
| | * | Fix variable definitionsGravatar MarkKoz2019-05-12-4/+4
| | | |
| | * | Build all images by defaultGravatar MarkKoz2019-05-12-7/+15
| | | |
| | * | Fall back to target branch if a commit can't be retrievedGravatar MarkKoz2019-05-12-4/+7
| | | |
| | * | Ensure branches are prefixed with refs/heads/Gravatar MarkKoz2019-05-12-0/+3
| | | |
| | * | Retrieve previous build's commit using a REST APIGravatar MarkKoz2019-05-12-8/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The "Builds - List" endpoint is used to retrieve only the most recent successful build. Builds will be fetched for the target branch if the current build is for a PR. Otherwise, the current build's source branch is used. * Remove variable group and COMMIT_HASH stuff * Add logging to the "Check Changed Files" step
| | * | Remove pushing images for testing CIGravatar MarkKoz2019-05-12-4/+3
| | | |
| | * | Use commit hash variable from variable groupGravatar MarkKoz2019-05-12-2/+8
| | | |
| | * | Azure: Add build and push steps for base and venv imagesGravatar MarkKoz2019-03-30-1/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The base image is only built when base.Dockerfile changes. The virtual environment image is only built when the base image is built, venv.Dockerfile changes, or either Pipfile changes. Images are pushed only after all images have been successfully built.