Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Re-enable golangci-lint #2130

Merged
merged 24 commits into from Sep 6, 2019

Conversation

@rafaelvanoni
Copy link
Member

rafaelvanoni commented Aug 30, 2019

No description provided.

Sync with upstream
@rafaelvanoni rafaelvanoni requested a review from projectcalico/core-maintainers as a code owner Aug 30, 2019
@rafaelvanoni rafaelvanoni self-assigned this Aug 30, 2019
calc/vxlan_resolver.go Outdated Show resolved Hide resolved
daemon/daemon.go Outdated Show resolved Hide resolved
dataplane/external/ext_dataplane.go Outdated Show resolved Hide resolved
dataplane/external/ext_dataplane.go Outdated Show resolved Hide resolved
fv/containers/containers.go Outdated Show resolved Hide resolved
hashutils/id.go Outdated Show resolved Hide resolved
iptables/rules.go Outdated Show resolved Hide resolved
iptables/rules.go Outdated Show resolved Hide resolved
k8sfv/calc_graph.go Outdated Show resolved Hide resolved
@rafaelvanoni rafaelvanoni requested a review from fasaxc Sep 4, 2019
Makefile Outdated Show resolved Hide resolved
bpf/bpf.go Outdated Show resolved Hide resolved
bpf/mock_bpf_lib.go Outdated Show resolved Hide resolved
dataplane/linux/ipip_mgr.go Show resolved Hide resolved
dataplane/mock/mock_dataplane.go Outdated Show resolved Hide resolved
iptables/rules.go Outdated Show resolved Hide resolved
k8sfv/k8sfv.go Outdated Show resolved Hide resolved
usagerep/usagerep_test.go Outdated Show resolved Hide resolved
daemon/daemon.go Show resolved Hide resolved
daemon/daemon.go Show resolved Hide resolved
@rafaelvanoni rafaelvanoni requested a review from fasaxc Sep 4, 2019
rafaelvanoni added 4 commits Sep 4, 2019
dataplane/mock/mock_dataplane.go Outdated Show resolved Hide resolved
Copy link
Member

fasaxc left a comment

Just a couple of nits

@rafaelvanoni rafaelvanoni requested a review from fasaxc Sep 5, 2019
@rafaelvanoni rafaelvanoni requested a review from fasaxc Sep 6, 2019
@fasaxc
fasaxc approved these changes Sep 6, 2019
Copy link
Member

fasaxc left a comment

One nit but feel free to fix up and merge

fv/test-workload/test-workload.go Outdated Show resolved Hide resolved
rafaelvanoni added 2 commits Sep 6, 2019
@rafaelvanoni rafaelvanoni merged commit dc02bc8 into projectcalico:master Sep 6, 2019
2 checks passed
2 checks passed
license/cla Contributor License Agreement is signed.
Details
semaphoreci The build passed on Semaphore.
Details
@rafaelvanoni rafaelvanoni deleted the rafaelvanoni:rafael-felix-3 branch Sep 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.