Skip to content

Add releaser#222

Merged
katcipis merged 6 commits intomasterfrom
addReleaser
May 11, 2017
Merged

Add releaser#222
katcipis merged 6 commits intomasterfrom
addReleaser

Conversation

@katcipis
Copy link
Copy Markdown
Member

Shifting from Full manual to Semi Auto =P. Not fully auto yet.

@katcipis katcipis self-assigned this May 11, 2017
@katcipis katcipis requested review from i4ki and vitorarins May 11, 2017 15:30
@codecov-io
Copy link
Copy Markdown

codecov-io commented May 11, 2017

Codecov Report

Merging #222 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #222   +/-   ##
=======================================
  Coverage   56.77%   56.77%           
=======================================
  Files          25       25           
  Lines        4067     4067           
=======================================
  Hits         2309     2309           
  Misses       1519     1519           
  Partials      239      239

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7b41ca0...ef3c16a. Read the comment docs.

supported_arch = ("amd64")

# Guarantee passing tests at least on the host arch/os
make test
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice =)

Copy link
Copy Markdown
Collaborator

@i4ki i4ki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

:)

@i4ki
Copy link
Copy Markdown
Collaborator

i4ki commented May 11, 2017

codecov is freaking out... No go files were touched.

@i4ki i4ki closed this May 11, 2017
@i4ki i4ki removed the in progress label May 11, 2017
@i4ki i4ki reopened this May 11, 2017
@i4ki
Copy link
Copy Markdown
Collaborator

i4ki commented May 11, 2017

sorry :( closed by mistake hahaha

@katcipis
Copy link
Copy Markdown
Member Author

auheuheuaheuaheuaheauh, will merge soon if @vitorarins does not make any movements

Copy link
Copy Markdown
Member

@vitorarins vitorarins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@vitorarins
Copy link
Copy Markdown
Member

We can add this: https://docs.travis-ci.com/user/deployment/releases/

in a later PR maybe

@katcipis katcipis merged commit 8ac7052 into master May 11, 2017
@katcipis katcipis deleted the addReleaser branch May 11, 2017 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants