aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * 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 #21 from python-discord/docstring-lint-changeGravatar Chris Goes2019-04-29-9/+1
| |\ | | | | | | Docstring lint change
| | * Remove space between ignored flake8 rulesGravatar Mark2019-04-24-1/+1
| | |
| | * Put back D400 as ignored codeGravatar sco12019-04-24-1/+1
| | | | | | | | | | | | Oops
| | * Relint Snekbox with new linting rulesGravatar sco12019-04-24-8/+0
| | |
| | * Remove docstring newline linting from ignored codesGravatar sco12019-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-fixGravatar Chris Goes2019-04-29-45/+39
| |\ \ | | |/ | |/| Manually pin urllib3 version to mitigate vulnerability
| | * Manually pin urllib3 version to mitigate vulnerabilityGravatar sco12019-04-21-45/+39
| |/
| * Merge pull request #18 from python-discord/update-contrib-docGravatar scragly2019-04-04-4/+57
| |\ | | | | | | | | | | | | Update contributor doc
| | * Update contributor docGravatar sco12019-04-03-4/+57
| |/
* | 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.
| * | Re-lock and update test job to Python 3.7Gravatar MarkKoz2019-03-29-11/+3
| | |
| * | Restructure Docker imagesGravatar MarkKoz2019-03-29-34/+40
|/ / | | | | | | | | | | | | | | * Create a separate image for the virtual environment * Build NsJail in the base image * Remove the NsJail binaries * Replace tini with Docker's init feature * Update Python to 3.7.3
* | Remove GitLab CI configGravatar MarkKoz2019-03-28-26/+0
| |
* | Merge pull request #15 from python-discord/refactor/restructureGravatar Mark2019-03-28-50/+50
|\ \ | | | | | | Restructure the Project Layout
| * | Restructure project layoutGravatar MarkKoz2019-03-28-50/+50
|/ / | | | | | | | | | | | | | | | | * Move all code into a "snekbox" package * Use logging code as __init__.py * Rename Snekbox class to NsJail * Create "site" sub-package * Move templates into this sub-package * Move Flask code into a new snekapp module
* | Replace RMQ with a POST endpoint (#7)Gravatar Mark2019-03-28-694/+146
|\ \ | |/ |/|
| * Merge remote-tracking branch 'origin' into rmq_removalGravatar MarkKoz2019-03-28-46/+76
| |\ | |/ |/|