hashgraph / hashgraph/solo-weaver

Replace duplicated phase enum in internal/consensus/phase.go with pkg/jobs types

Open
#863 0 comments 0 reactions 0 assignees View on GitHub
Improvement Story
Dominant language
Go
Stars
3
Forks
0
Avg merge
3d 2h
Merged PRs (30d)
46

Description

**Epic:** #1081

`internal/consensus/phase.go` currently duplicates phase name constants that are now defined in `pkg/jobs`. Replace the local definitions with imports from `github.com/hashgraph/solo-operator/pkg/jobs`.

## Acceptance Criteria
- [ ] Local phase enum constants removed from `internal/consensus/phase.go`
- [ ] All callers updated to use `jobs.PhaseUpgradePrepare` etc.
- [ ] No behaviour change — pure refactor

Contributor guide

Open the contributing guide

Research direction

Start with internal/consensus/phase.go and inspect the phase constants currently duplicated there. Search for callers of those local constants, update them to use the corresponding pkg/jobs types such as jobs.PhaseUpgradePrepare, and verify that the local enum is gone with no behavior changes.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
backend
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.