Skip to content

feat: rename RepositoryConfig.weight to emission_share (#1215)#1241

Closed
alpurkan17 wants to merge 8 commits into
entrius:testfrom
alpurkan17:feat/1215-emission-share
Closed

feat: rename RepositoryConfig.weight to emission_share (#1215)#1241
alpurkan17 wants to merge 8 commits into
entrius:testfrom
alpurkan17:feat/1215-emission-share

Conversation

@alpurkan17
Copy link
Copy Markdown
Contributor

Summary

Rename RepositoryConfig.weight to emission_share as step 1 of #1215.

  • Field renamed in dataclass + all code references
  • Function resolve_repo_weightresolve_emission_share
  • Constant DEFAULT_REPO_WEIGHTDEFAULT_EMISSION_SHARE
  • JSON parser reads emission_share key

Part of #1215

@xiao-xiao-mao xiao-xiao-mao Bot added the refactor Code restructuring without behavior change label May 13, 2026
@alpurkan17 alpurkan17 force-pushed the feat/1215-emission-share branch 4 times, most recently from e68078b to 8179d0f Compare May 13, 2026 06:56
@alpurkan17 alpurkan17 force-pushed the feat/1215-emission-share branch 2 times, most recently from 65c538a to 7dc8a30 Compare May 13, 2026 07:08
@alpurkan17 alpurkan17 force-pushed the feat/1215-emission-share branch from 7dc8a30 to 52b431f Compare May 13, 2026 07:10
@alpurkan17 alpurkan17 force-pushed the feat/1215-emission-share branch from 5fbcba9 to efc8287 Compare May 13, 2026 07:17
@alpurkan17 alpurkan17 force-pushed the feat/1215-emission-share branch 3 times, most recently from cc0372b to 6099a47 Compare May 13, 2026 07:29
@alpurkan17 alpurkan17 force-pushed the feat/1215-emission-share branch from 6099a47 to b820d20 Compare May 13, 2026 07:31
@alpurkan17 alpurkan17 force-pushed the feat/1215-emission-share branch 3 times, most recently from cadd39e to c6315ac Compare May 13, 2026 07:48
@alpurkan17 alpurkan17 force-pushed the feat/1215-emission-share branch from c6315ac to a55a3ab Compare May 13, 2026 07:52
@alpurkan17
Copy link
Copy Markdown
Contributor Author

Ready for review. All 11 deliverables from the issue are implemented, CI passes, and the two blockers raised on the competing PR (case sensitivity and collateral subtraction) are already addressed in this implementation.

@alpurkan17
Copy link
Copy Markdown
Contributor Author

@anderdc @LandynDev PR #1241 ready for review. All 11 deliverables from #1215 implemented, CI passes, and the two blockers identified on the competing PR (case sensitivity and collateral subtraction) are already addressed.

@alpurkan17
Copy link
Copy Markdown
Contributor Author

Competitor #1235 sudah merged. Closing.

@alpurkan17 alpurkan17 closed this May 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Code restructuring without behavior change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant