erigontech / erigontech/erigon

Refactor QA workflows

Open
#15,610 1 comment 0 reactions 0 assignees View on GitHub
QA
Dominant language
Go
Stars
3.6k
Forks
1.5k
Avg merge
1d 16h
Merged PRs (30d)
455

Description

The QA workflows are reused with the same pattern in numerous workflows. The one I encountered was tip-tracking:

```
> rg -l -. 'qa-tests/tip-tracking' | xargs basename
qa-sync-test-bisection-tool.yml
qa-tip-tracking.yml
qa-tip-tracking-polygon.yml
qa-sync-from-scratch.yml
qa-constrained-tip-tracking.yml
qa-sync-with-externalcl.yml
qa-tip-tracking-gnosis.yml
release.yml
qa-sync-from-scratch-minimal-node.yml
```

Adding a [change](https://github.com/erigontech/erigon/pull/15292) to the QA required editing all the workflows and is potentially very error prone. I think we could avoid some duplication by defining some common steps/workflow such as in https://docs.github.com/en/actions/sharing-automations/avoiding-duplication with variables/inputs to handle the small variations.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.