Skip to content

Update submodules#27

Merged
dagacha merged 3 commits into
mainfrom
chore/update_toolkit_submodules
Feb 24, 2023
Merged

Update submodules#27
dagacha merged 3 commits into
mainfrom
chore/update_toolkit_submodules

Conversation

@jmoreira-valory
Copy link
Copy Markdown
Contributor

@jmoreira-valory jmoreira-valory commented Feb 24, 2023

DOC SUBMODULE UPDATE PR

Please, each PO approve this PR after confirming that for the submodule linked to their repos:

  1. The submodule update is pointing to the intended commit, which should be the latest tag/release.

  2. To the best of your knowledge, the public-facing tutorial in the submodule (<submodule_folder>/docs/index.md) is up to date. In particular, you agree with the instructions in the section "Demo".

  3. You have replicated the instructions exactly as described in the public-facing tutorial (with the exact options, hash values, command line parameters, private keys, etc. described there), in a clean virtual environment, and it has worked. Also:
    a. an external user is able to replicate the demo with these instructions, or

    b. if this is not 100% possible, (e.g., due to requiring private API keys, registered addresses, etc) this limitation is clearly informed to the user in the tutorial.

@dagacha dagacha merged commit 061760a into main Feb 24, 2023
Copy link
Copy Markdown
Contributor

@Adamantios Adamantios left a comment

Choose a reason for hiding this comment

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

LGTM

@jmoreira-valory jmoreira-valory deleted the chore/update_toolkit_submodules branch September 7, 2023 08:29
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.

4 participants