CI and Linter #27

2020-01-25T04:59:19Z

jolheiser commented

2020-01-25 04:59:19 +00:00

(Migrated from gitea.com)

This PR adds a Drone pipeline for releasing on new tags, as well as running tests in all directories and building on each push to master (to ensure builds succeed).
It also adds a linter (and some code changes to appease said linter).

Note: The pipeline to test builds is currently not restricted to master while this PR is WIP in order for me to make any necessary changes. Once it succeeds, I will commit a change to lock building to master.
https://drone.gitea.com/gitea/changelog/18/2/3

This PR adds a Drone pipeline for releasing on new tags, as well as running tests in _all_ directories and building on each push to `master` (to ensure builds succeed). It also adds a linter (and some code changes to appease said linter). ~~**Note:** The pipeline to test builds is currently not restricted to `master` while this PR is WIP in order for me to make any necessary changes. Once it succeeds, I will commit a change to lock building to `master`.~~ https://drone.gitea.com/gitea/changelog/18/2/3

jolheiser (Migrated from gitea.com) reviewed 2020-01-25 05:21:21 +00:00

techknowlogick (Migrated from gitea.com) reviewed 2020-01-25 05:31:32 +00:00

lunny (Migrated from gitea.com) approved these changes 2020-01-26 03:14:01 +00:00

6543 (Migrated from gitea.com) approved these changes 2020-01-26 03:22:05 +00:00

6543 (Migrated from gitea.com) left a comment

is token added ?

beside lgtm

is token added ? beside lgtm

jolheiser commented

2020-01-26 03:33:23 +00:00

(Migrated from gitea.com)

Reminder not to merge until all needed secrets are added. 😄