Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Renamed STRK to FRI #570

Merged
merged 3 commits into from
Aug 2, 2024
Merged

Renamed STRK to FRI #570

merged 3 commits into from
Aug 2, 2024

Conversation

marioiordanov
Copy link
Contributor

@marioiordanov marioiordanov commented Jul 30, 2024

Usage related changes

CLI arguments changes:

gas-price-strk -> gas-price-fri
data-gas-price-strk -> data-gas-price-fri

GAS_PRICE_STRK -> GAS_PRICE_FRI
DATA_GAS_PRICE_STRK -> DATA_GAS_PRICE_FRI

Devnet methods:

In json request/response:
gas_price_strk -> gas_price_fri
data_gas_price_strk -> data_gas_price_fri

Development related changes

Renamed STRK to FRI

Checklist:

  • Checked out the contribution guidelines
  • Applied formatting - ./scripts/format.sh
  • No linter errors - ./scripts/clippy_check.sh
  • No unused dependencies - ./scripts/check_unused_deps.sh
  • No spelling errors - ./scripts/check_spelling.sh
  • Performed code self-review
  • Rebased to the latest commit of the target branch (or merged it into my branch)
    • Once you make the PR reviewable, please avoid force-pushing
  • Updated the docs if needed - ./website/README.md
  • Linked the issues resolvable by this PR - linking info
  • Updated the tests if needed; all passing - execution info

@marioiordanov marioiordanov marked this pull request as ready for review July 31, 2024 11:13
@marioiordanov marioiordanov linked an issue Jul 31, 2024 that may be closed by this pull request
@marioiordanov marioiordanov merged commit 6d55097 into main Aug 2, 2024
@marioiordanov marioiordanov deleted the strk_to_fri branch August 2, 2024 08:58
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.

Unification of wei and fri
2 participants