Skip to content
This repository has been archived by the owner on Jun 23, 2020. It is now read-only.

Use standard drone config similar to other drone libs #50

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Use standard drone config similar to other drone libs #50

wants to merge 1 commit into from

Conversation

tboerger
Copy link

No description provided.

@CLAassistant
Copy link

CLAassistant commented Feb 20, 2019

CLA assistant check
All committers have signed the CLA.

.drone.yml Outdated Show resolved Hide resolved
.drone.yml Outdated Show resolved Hide resolved
.drone.yml Outdated Show resolved Hide resolved
@bradrydzewski
Copy link
Member

bradrydzewski commented Feb 20, 2019

I think we could simplify to this:

---
kind: pipeline
name: testing

platform:
  os: linux
  arch: amd64

steps:
- name: test
  image: golang:1.11
  commands:
  - go vet ./...
  - go test -cover ./...

I know the plugins follow different conventions, but the plugins tend to have very complex pipelines (multi-os, multi-arch, docker builds, etc). For libraries I think we can keep things simple.

@tboerger
Copy link
Author

Not exactly like you defined it, but I have simplified the pipeline now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants