From a5168722b9dca079fa6f2b9be4725366282e351c Mon Sep 17 00:00:00 2001 From: Charles Francoise Date: Tue, 5 Sep 2023 19:58:25 +0200 Subject: [PATCH] revert: "feat: Put back telemetry" (#1001) This reverts commit 4be7e3abe5032fdfe49af4272757e4b97bc01be5. --- .circleci/config.yml | 6 - Makefile | 2 +- README.md | 20 +- Taskfile.yml | 10 +- api/api.go | 19 -- clitest/clitest.go | 22 +- clitest/telemetry.go | 19 -- cmd/build.go | 10 +- cmd/cmd_suite_test.go | 3 - cmd/completion.go | 10 +- cmd/config.go | 31 +-- cmd/config_test.go | 32 +-- cmd/create_telemetry.go | 206 ------------------- cmd/create_telemetry_test.go | 321 ----------------------------- cmd/diagnostic.go | 12 +- cmd/diagnostic_test.go | 19 -- cmd/follow.go | 12 +- cmd/info/info.go | 10 +- cmd/info/info_test.go | 58 +----- cmd/namespace.go | 10 +- cmd/namespace_test.go | 37 ---- cmd/open.go | 17 +- cmd/orb.go | 6 - cmd/orb_test.go | 52 ----- cmd/policy/policy.go | 18 +- cmd/policy/policy_test.go | 11 +- cmd/root.go | 34 +--- cmd/root_test.go | 8 +- cmd/runner/instance.go | 13 +- cmd/runner/resource_class.go | 7 - cmd/runner/telemetry_test.go | 62 ------ cmd/runner/token.go | 7 - cmd/setup.go | 9 +- cmd/setup_test.go | 32 --- cmd/telemetry.go | 79 -------- cmd/telemetry_test.go | 58 ------ cmd/telemetry_unit_test.go | 143 ------------- cmd/update.go | 17 +- cmd/update_test.go | 65 ------ cmd/version.go | 14 +- cmd/version_test.go | 39 ---- go.mod | 5 - go.sum | 377 ----------------------------------- main.go | 6 +- prompt/prompt.go | 11 - settings/settings.go | 84 ++------ telemetry/context.go | 16 -- telemetry/events.go | 149 -------------- telemetry/telemetry.go | 197 ------------------ telemetry/utils.go | 31 --- 50 files changed, 90 insertions(+), 2346 deletions(-) delete mode 100644 clitest/telemetry.go delete mode 100644 cmd/create_telemetry.go delete mode 100644 cmd/create_telemetry_test.go delete mode 100644 cmd/runner/telemetry_test.go delete mode 100644 cmd/telemetry.go delete mode 100644 cmd/telemetry_test.go delete mode 100644 cmd/telemetry_unit_test.go delete mode 100644 cmd/version_test.go delete mode 100644 telemetry/context.go delete mode 100644 telemetry/events.go delete mode 100644 telemetry/telemetry.go delete mode 100644 telemetry/utils.go diff --git a/.circleci/config.yml b/.circleci/config.yml index 8a1b5456c..edd87a19d 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -137,10 +137,6 @@ jobs: working_directory: integration_tests environment: TESTING: "true" - - run: - name: "Make sure simple command do not cause any timeout" - command: circleci version - test: executor: go steps: @@ -382,14 +378,12 @@ workflows: - devex-release - deploy: requires: - - cucumber - test - test_mac - coverage - lint - deploy-test - shellcheck/check - - vulnerability-scan filters: branches: only: main diff --git a/Makefile b/Makefile index a51a58c4a..604d50148 100644 --- a/Makefile +++ b/Makefile @@ -4,7 +4,7 @@ GOOS=$(shell go env GOOS) GOARCH=$(shell go env GOARCH) build: always - go build -o build/$(GOOS)/$(GOARCH)/circleci -ldflags='-X github.com/CircleCI-Public/circleci-cli/telemetry.SegmentEndpoint=https://api.segment.io' + go build -o build/$(GOOS)/$(GOARCH)/circleci build-all: build/linux/amd64/circleci build/darwin/amd64/circleci diff --git a/README.md b/README.md index b57b73873..a955cff70 100644 --- a/README.md +++ b/README.md @@ -180,20 +180,6 @@ Please see the [documentation](https://circleci-public.github.io/circleci-cli) o ## Server compatibility -| Functionality | Impacted commands | Change description | Compatibility with Server | -| --- | --- | --- | --- | -| Config compilation and validation |