pokt-network / poktroll

The official Shannon upgrade implementation of the Pocket Network Protocol implemented using the Cosmos SDK
MIT License
15 stars 8 forks source link

[Tooling, Code Health] refactor: split Makefile into multiple files #816

Closed bryanchriswhite closed 1 month ago

bryanchriswhite commented 1 month ago

Summary

The Makefile has gotten quite large (1185 LoC). This PR splits the makefile into multiple files, focusing on large groups of targets and groups related to on-chain state or local/devnets:

Issue

Type of change

Select one or more from the following:

Testing

Sanity Checklist

github-actions[bot] commented 1 month ago

The CI will now also run the e2e tests on devnet, which increases the time it takes to complete all CI checks.

You may need to run make trigger_ci to submit an empty commit that'll trigger the tests.

GCP workloads (requires changing the namespace to 816)) Grafana network dashboard for devnet-issue-816

bryanchriswhite commented 1 month ago

Merging despite CI failures.

  1. CI passed as of 335e68b
  2. Subsequent commits only "merge with main", reverting commits that were merged from another branch into this branches original base (the PR base has since been updated to main).
  3. This branch only modifies the makefiles image