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

Testing smart contracts #1418

Merged
merged 2 commits into from
Apr 16, 2024
Merged

Conversation

sczembor
Copy link
Contributor

What does this PR fix/introduce?

  • Update the code examples to use the latest versions of dependencies and replace DEFAULT_RUN_GENESIS_REQUEST with PRODUCTION_RUN_GENESIS_REQUEST (The counter code was already updated)
  • Add a tip section where testing time sensitive functions in a smart contract is explained
  • Add Adding New Fields During Upgrades section where adding new named_keys to the smart contract on update is explained.
  • Update the docs to list the v3 version of the counter that is introduced in this PR Counter-v3 casper-ecosystem/counter#31
  • Fix formatting

Reviewers

…rial to include the counter-v3; add a tip section on how to test time sensitive functions in smart contract; add a section on how to add new named_keys when updating the smart_contract
@sczembor sczembor requested a review from ACStone-MTS as a code owner March 20, 2024 15:43
Copy link
Contributor

@ACStone-MTS ACStone-MTS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sczembor sczembor merged commit a90f97d into casper-network:dev Apr 16, 2024
8 checks passed
@sczembor sczembor deleted the testing_smart_contracts branch April 16, 2024 12:25
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