Skip to content

chore: ADR 109: go mod tidy#1606

Merged
thanethomson merged 3 commits intofeature/adr109-reduce-go-api-surfacefrom
thane/adr109/fix-gomod
Nov 13, 2023
Merged

chore: ADR 109: go mod tidy#1606
thanethomson merged 3 commits intofeature/adr109-reduce-go-api-surfacefrom
thane/adr109/fix-gomod

Conversation

@thanethomson
Copy link
Contributor

@thanethomson thanethomson commented Nov 13, 2023

CodeQL seems to be complaining about some recent changes that go mod tidy produces.

Seems to be fixed after removing the patch version numbers in the go.mod files: https://github.com/cometbft/cometbft/actions/runs/6852623623/job/18631585336


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
@thanethomson thanethomson marked this pull request as ready for review November 13, 2023 16:14
@thanethomson thanethomson requested a review from a team as a code owner November 13, 2023 16:14
@thanethomson thanethomson requested a review from a team November 13, 2023 16:14
@thanethomson thanethomson merged commit 51d0865 into feature/adr109-reduce-go-api-surface Nov 13, 2023
@thanethomson thanethomson deleted the thane/adr109/fix-gomod branch November 13, 2023 16:48
github-merge-queue bot pushed a commit that referenced this pull request Nov 15, 2023
* ADR 109: Internalize specific packages to reduce surface area (#1485)

* consensus: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* evidence: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* inspect: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* state: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* blocksync: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* statesync: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* store: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/async: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/autofile: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/bits: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/clist: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/cmap: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/events: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/fail: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/flowrate: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/os: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/progressbar: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/protoio: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/pubsub: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/net: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/rand: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/service: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/strings: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/sync: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/tempfile: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* libs/timer: Move into internal

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Add changelog entries

Signed-off-by: Thane Thomson <connect@thanethomson.com>

---------

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* ADR 109: Modularize test infra (#1488)

* test/e2e: Split out as separate module

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* test/loadtime: Split out as separate module

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* test/e2e: Remove optimization from Docker image construction

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Ensure that linter covers E2E framework and app

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Update CI linting to cover submodules

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Add changelog entry

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Expand linter coverage to loadtime tool

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Add missing phony entries

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* test/e2e: Sync debug Dockerfile with primary Dockerfile

Signed-off-by: Thane Thomson <connect@thanethomson.com>

---------

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* chore: ADR 109: go mod tidy (#1606)

* go mod tidy

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* go.mod: Remove patch version

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* go.mod: Remove new toolchain directives

Signed-off-by: Thane Thomson <connect@thanethomson.com>

---------

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* ADR 109: Fix mock generation (#1607)

* internal: Fix mockery code generation script paths

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* make mockery

Signed-off-by: Thane Thomson <connect@thanethomson.com>

---------

Signed-off-by: Thane Thomson <connect@thanethomson.com>

---------

Signed-off-by: Thane Thomson <connect@thanethomson.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants