Skip to content

Refactor: unify config loading by reusing load_config from helpers#410

Closed
volcano303 wants to merge 1 commit intoentrius:testfrom
volcano303:refactor/unify-config-loading
Closed

Refactor: unify config loading by reusing load_config from helpers#410
volcano303 wants to merge 1 commit intoentrius:testfrom
volcano303:refactor/unify-config-loading

Conversation

@volcano303
Copy link
Copy Markdown

@volcano303 volcano303 commented Apr 14, 2026

Summary

  • Replace the standalone _load_config_value() in cli/miner_commands/post.py with a call to the existing load_config() from cli/issue_commands/helpers.py
  • Replace the standalone get_contract_address() in validator/utils/issue_competitions.py with a call to the existing get_contract_address() from cli/issue_commands/helpers.py
  • Same file paths, same error handling, just one implementation now

Move config loading in post.py and contract address resolution in
issue_competitions.py to use centralized helpers, eliminating duplicated
logic and ensuring consistent priority (CLI arg > env var > default).
@volcano303 volcano303 force-pushed the refactor/unify-config-loading branch from 82f428e to 82e39f9 Compare April 14, 2026 04:51
@anderdc anderdc closed this Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants