Skip to content

Commit

Permalink
add sqflint to circleci (#6715)
Browse files Browse the repository at this point in the history
* add sqflint to circleci

* lint working directory

* do not exit with 1 on errors

* split up build jobs

* change linting not to be required

* use armake docker image for updating docs
  • Loading branch information
thojkooi authored Dec 4, 2018
1 parent 24e1bff commit 30769f9
Showing 1 changed file with 41 additions and 4 deletions.
45 changes: 41 additions & 4 deletions circle.yml
Original file line number Diff line number Diff line change
@@ -1,26 +1,63 @@
version: 2
jobs:
build:
validate-scripts:
docker:
- image: acemod/armake:master
- image: acemod/sqflint:latest
steps:
- checkout
- run:
name: Validate SQF and Config style
command: python tools/sqf_validator.py && python tools/config_style_checker.py

linting:
docker:
- image: acemod/sqflint:latest
steps:
- checkout
- run:
name: Lint sqf code
command: sqflint -d addons || true

armake:
docker:
- image: acemod/armake:master
steps:
- checkout
- run:
name: Version
command: armake --version
- run:
name: Build
command: |
make -j 4
- deploy:
update-docs:
docker:
- image: acemod/armake:latest
steps:
- checkout
- run:
name: Update documentation and translation statistics
command: |
command: |
if [ "${CIRCLE_BRANCH}" == "master" ] && [ "${CIRCLE_PROJECT_USERNAME}" == "acemod" ]; then
pip install pygithub pygithub3
python3 tools/deploy.py
else
echo "Skipping, not on acemod/ACE3 master branch..."
fi
workflows:
version: 2
build-job:
jobs:
- linting
- validate-scripts
- armake:
requires:
- validate-scripts
- update-docs:
requires:
- armake
filters:
branches:
only: master

0 comments on commit 30769f9

Please sign in to comment.