Joystream / Joystream/joystream

Hotfix: Consolidate parameters in one `constants.rs`

Open
#4,222 0 comments 0 reactions 0 assignees View on GitHub
post-mainnet runtime tech-debt
Dominant language
Rust
Stars
1.4k
Forks
116
PR merge metrics
No merged PRs in 30d

Description

# Background

Parameter constants, with values set at compile time, are distributed across a variety of files in the runtime, and are hard to manage. Currently, constant parameters are distributed a variety of files, and intermingled with lots of other kinds of code, specifically files

* `lib.rs`
* `constants.rs`
* `proposal_configurations/mod.rs`
* `lock.rs`

# Proposal

At some point, we are going to just go full-on mutables, as is described here: https://github.com/Joystream/joystream/issues/4225, but I suspect that will take some more time to pull off, as it will have cascading consequences on other apps as well, so before that, lets just consolidate.

┆Issue is synchronized with this [Asana task](https://app.asana.com/0/1201958687417145/1202872445073944) by [Unito](https://www.unito.io)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.