Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

Document valid address #428

Merged
merged 2 commits into from
Nov 30, 2022
Merged

Document valid address #428

merged 2 commits into from
Nov 30, 2022

Conversation

merdoyovski
Copy link
Contributor

Closes #419

  • Added an admonition for documenting what a valid address is.

@merdoyovski merdoyovski requested a review from orkunkl November 29, 2022 22:12
@@ -250,9 +256,9 @@ You get a lot of flexibility when you are defining your own mocked contract. You

Different chains and hubs in the Cosmos ecosystem may have some variations on how migrations are done on their respective networks. This section will attempt to outline those.

### Juno
### Juno
Copy link
Contributor

Choose a reason for hiding this comment

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

Back then it made sense to give references to active cosmwasm projects. But now there are many chains so it makes most sense to not give references. @merdoyovski can you change this with another PR?

@orkunkl orkunkl merged commit 93d5b95 into main Nov 30, 2022
@orkunkl orkunkl deleted the testing branch November 30, 2022 08:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unit testing: document what's a valid address
2 participants