Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

Remove codecov/project step #629

Merged

Conversation

victorschumacherzup
Copy link
Contributor

@victorschumacherzup victorschumacherzup commented Oct 22, 2020

Description

The codecov/project check is no longer making sense
We need to constantly update the branch because master is always changing

image

Changelog

Disable codecov/project check

Signed-off-by: victor-schumacher <victorschumacher255@gmail.com>
@codecov-io
Copy link

codecov-io commented Oct 22, 2020

Codecov Report

Merging #629 into master will decrease coverage by 1.30%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #629      +/-   ##
==========================================
- Coverage   81.41%   80.11%   -1.31%     
==========================================
  Files         103       99       -4     
  Lines        3643     3299     -344     
==========================================
- Hits         2966     2643     -323     
+ Misses        483      467      -16     
+ Partials      194      189       -5     
Impacted Files Coverage Δ
pkg/cmd/cmd.go 71.42% <0.00%> (-8.58%) ⬇️
pkg/formula/runner/local/pre_run.go 90.47% <0.00%> (-3.18%) ⬇️
pkg/formula/formula.go 80.00% <0.00%> (-0.77%) ⬇️
pkg/formula/runner.go 0.00% <0.00%> (ø)
pkg/formula/input/prompt/prompt.go
pkg/formula/input/stdin/stdin.go
pkg/formula/runner/input_resolver.go
pkg/formula/input/flag/flag.go
pkg/formula/runner/docker/runner.go 92.45% <0.00%> (+0.56%) ⬆️
pkg/formula/runner/local/runner.go 94.73% <0.00%> (+1.51%) ⬆️
... and 1 more

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 b7a0546...61f70e2. Read the comment docs.

Signed-off-by: victor-schumacher <victorschumacher255@gmail.com>
Signed-off-by: victor-schumacher <victorschumacher255@gmail.com>
@victorschumacherzup victorschumacherzup marked this pull request as ready for review October 22, 2020 12:51
@victorschumacherzup victorschumacherzup self-assigned this Oct 22, 2020
@victorschumacherzup victorschumacherzup added the 🔨 improvement Improvement in features label Oct 22, 2020
@victorschumacherzup victorschumacherzup changed the title remove codecov project remove codecov/project step Oct 22, 2020
Signed-off-by: victor-schumacher <victorschumacher255@gmail.com>
@victorschumacherzup victorschumacherzup added the 🚧 WIP Work in Progress label Oct 22, 2020
@brunasilvazup brunasilvazup changed the title remove codecov/project step Remove codecov/project step Oct 22, 2020
@brunasilvazup brunasilvazup added ✔️ ready-for-review ready for review and removed 🚧 WIP Work in Progress labels Oct 22, 2020
Signed-off-by: victor-schumacher <victorschumacher255@gmail.com>
Copy link
Contributor

@henriquemoraeszup henriquemoraeszup left a comment

Choose a reason for hiding this comment

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

YES

@@ -180,33 +180,7 @@ func TestPreRun(t *testing.T) {
err: errors.New("error to create dir"),
},
},
{
name: "copy work dir error",
Copy link
Contributor

Choose a reason for hiding this comment

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

what is this change for?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

just removing it to test the codecov, i will put it back after finishing

Copy link
Contributor Author

@victorschumacherzup victorschumacherzup Oct 22, 2020

Choose a reason for hiding this comment

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

i request reviewers too early, sry

Copy link
Contributor

Choose a reason for hiding this comment

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

no worries
=]

Signed-off-by: victor-schumacher <victorschumacher255@gmail.com>
@henriquemoraeszup
Copy link
Contributor

I like how the explanation is phrased under Description 😃

@victorschumacherzup victorschumacherzup merged commit 65bf93b into ZupIT:master Oct 22, 2020
@victorschumacherzup victorschumacherzup deleted the fix/codeCovConfig branch October 22, 2020 16:55
brunonmelo pushed a commit to brunonmelo/ritchie-cli that referenced this pull request Nov 12, 2020
* remove codecov project

* ignore vendor

Signed-off-by: victor-schumacher <victorschumacher255@gmail.com>

Co-authored-by: victor-schumacher <victorschumacher255@gmail.com>
Signed-off-by: Bruno N. Melo <brunonobrega.melo@gmail.com>
brunonmelo pushed a commit to brunonmelo/ritchie-cli that referenced this pull request Nov 12, 2020
* remove codecov project

* ignore vendor

Signed-off-by: victor-schumacher <victorschumacher255@gmail.com>

Co-authored-by: victor-schumacher <victorschumacher255@gmail.com>
Signed-off-by: Bruno N. Melo <brunonobrega.melo@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🔨 improvement Improvement in features ✔️ ready-for-review ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants