Skip to content

Commit

Permalink
Add Go module support for pipelines.
Browse files Browse the repository at this point in the history
This enables Go modules and removes the vendor directory from the
repository.

NOTE: highly recommend viewing this change with
`git diff master -- ':!third_party' ':!vendor'`

Notable changes:
- Removes use of dep in favor of Go modules.
- Removes vendor directory. As a result, switches license notice
  generation to go-license.
- Forks code generation scripts from respective k8s/knative sources to
  allow module aware imports. (I want to fix this upstream sometime in
  the future, but for now this is as best as we can do).
- Unsymlinks config/300-imagecache.yaml, since go mod vendor will only
  copy package dependencies.
- Wraps `logger.Sync()` calls in func that explicitly ignores the
  returned error to make golangci-lint happy.
- Replace kodata/VENDOR-LICENSE symlinks with symlink to third_party.

This should not result in any change of functionality.

Fixes #1315, #1538
  • Loading branch information
wlynch authored and tekton-robot committed Dec 13, 2019
1 parent 60d0276 commit 5830188
Show file tree
Hide file tree
Showing 2,210 changed files with 355,922 additions and 258,702 deletions.
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@
*.dll

# Fortran module files
*.mod
*.smod

# Compiled Static libraries
Expand Down
274 changes: 132 additions & 142 deletions DEVELOPMENT.md

Large diffs are not rendered by default.

Loading

0 comments on commit 5830188

Please sign in to comment.