add ci context docker deployment update api docs add some small fixes
3.6 KiB
Pull Requests
When creating a pull-request you should:
- Open an issue first: Confirm that the change or feature will be accepted
- gofmt and vet the code: Use
gofmt
,golint
,govet
andgoimports
to clean up your code. - vendor dependencies with dep: Use
dep ensure --update
if you have added to or updated dependencies, so that they get added to the dependency manifest. - Update api documentation: If your pull-request adding/modifying an API request, make sure you update the swagger documentation (
api-docs.yml
) - Run Api Tests: If your pull request modifies the API make sure you run the integration tests using dredd.
Installation in a development environment
- Check out the
develop
branch - Install Go. Go must be >= v1.10 for all the tools we use to work
- Install MySQL / MariaDB
- Install node.js
- Set up GOPATH, GOBIN and Workspace.
cd {WORKING_DIRECTORY}
# Exports only needed pre Go 1.8 or for custom GOPATH location
export GOPATH=`pwd`
export GOBIN=$GOPATH/bin
export PATH=$PATH:$GOBIN
mkdir -p $GOPATH/src/github.com/ansible-semaphore && cd $GOPATH/src/github.com/ansible-semaphore
- Clone semaphore (with submodules)
git clone --recursive git@github.com:ansible-semaphore/semaphore.git && cd semaphore
- Install dev dependencies
go get -u github.com/go-task/task/cmd/task
task deps
Windows users will additionally need to manually install goreleaser from https://github.com/goreleaser/goreleaser/releases
- Set up config, database & run migrations
cat <<EOT >> config.json
{
"mysql": {
"host": "127.0.0.1:3306",
"user": "root",
"pass": "",
"name": "semaphore"
},
"port": ":3000"
}
EOT
echo "create database semaphore;" | mysql -uroot -p
task compile
go run cli/main.go -config ./config.json -migrate
Now it's ready to start.. Run task watch
- Watches js files in
public/js/*
and compiles into a bundle - Watches css files in
public/css/*
and compiles into css code - Watches pug files in
public/html/*
and compiles them into html - Watches go files and recompiles the binary
- Open localhost:3000
Note: for Windows, you may need Cygwin to run certain commands because the reflex package probably doesn't work on Windows.
You may encounter issues when running task watch
, but running task build
etc... will still be OK.
Integration Tests
Dredd is used for API integration tests, if you alter the API in any way you must make sure that the information in the api docs matches the responses.
As Dredd and the application database config may differ it expects it's own config.json in the .dredd folder. The most basic configuration for this using a local docker container to run the database would be
{
"mysql": {
"host": "0.0.0.0:3306",
"user": "semaphore",
"pass": "semaphore",
"name": "semaphore"
}
}
It is strongly advised to run these tests inside docker containers, as dredd will write a lot of test information and will NOT clear it up. This means that you should never run these tests against your productive database! The best practice to run these tests is to use docker and the task commands.
task dc:build #First run only to build the images
context=dev task dc:up
task test:api
# alternatively if you want to run dredd in a container use the following command.
# You will need to use the host network so that it can reach the docker container on a 0.0.0.0 address
# docker run -it --rm -v $(pwd):/home/developer/src --network host tomwhiston/dredd --config .dredd/dredd.yml