Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

incorporate latest boilerplate changes #53

Merged
merged 7 commits into from
Jan 24, 2020

Conversation

honnix
Copy link
Member

@honnix honnix commented Jan 21, 2020

This reflects changes done in flyteorg/boilerplate#4

@codecov-io
Copy link

codecov-io commented Jan 21, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   72.21%   72.21%           
=======================================
  Files          56       56           
  Lines        2242     2242           
=======================================
  Hits         1619     1619           
  Misses        500      500           
  Partials      123      123

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 ee0dc82...f11b511. Read the comment docs.

@@ -22,7 +22,6 @@ go_install_tool () {
# List of tools to go get
# In the format of "<cli>:<package>" or ":<package>" if no cli
tools=(
"pflags:github.com/lyft/flytestdlib/cli/pflags"
Copy link
Member Author

Choose a reason for hiding this comment

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

This doesn't work. I guess due to the funky dependency of itself.

Makefile Outdated Show resolved Hide resolved
@honnix honnix force-pushed the update-boilerplate branch from 6fe2ce6 to f11b511 Compare January 24, 2020 07:47
@honnix
Copy link
Member Author

honnix commented Jan 24, 2020

Not sure lyft/golangci_file is not there. I removed it manually after make update_boilerplate. If we need it that can be a separated PR.

@honnix
Copy link
Member Author

honnix commented Jan 24, 2020

@wild-endeavor PTAL

@wild-endeavor wild-endeavor merged commit e24904c into flyteorg:master Jan 24, 2020
@honnix honnix deleted the update-boilerplate branch January 28, 2020 06:17
eapolinario pushed a commit that referenced this pull request Sep 6, 2023
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.

4 participants