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

docs: Chainstack implementation update #4

Closed
wants to merge 12 commits into from
Closed

docs: Chainstack implementation update #4

wants to merge 12 commits into from

Conversation

soos3d
Copy link
Contributor

@soos3d soos3d commented Jul 28, 2022

What I did

  1. Updated README with clear instructions.
  2. Renamed provider.py to providers.py.
  3. Added Goerli network to __init__.py.

How I did it

Forked and modified my branch.
Ran pre-commit

Checklist

  • [ x] Passes all linting checks (pre-commit and CI jobs)
  • [x ] New test cases have been added and are passing
  • [x ] Documentation has been updated
  • [ x] PR title follows Conventional Commit standard (will be automatically included in the changelog)

Co-authored-by: NotPeopling2day <[email protected]>
@NotPeopling2day
Copy link
Contributor

I updated the base code so that it was a little more functional and added a couple tests. Unfortunately it will require you to rebase/fix conflicts.


```bash
ape console --network ethereum:ropsten:chainstack
ape console --network chainstack
Copy link
Member

Choose a reason for hiding this comment

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

You can see the complete list by doing ape networks list. I think we should document it that way instead because it does not then require putting in an errornous command.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My idea with this is to show the user the syntax to use and all the different options. But I'm ok removing that part if you prefer, the part above it shows the ape networks list command anyway 😄

soos3d and others added 2 commits August 1, 2022 15:53
Co-authored-by: Juliya Smith <[email protected]>
Co-authored-by: Juliya Smith <[email protected]>
@soos3d
Copy link
Contributor Author

soos3d commented Aug 2, 2022

I think I'll close this PR and re-fork the updated code. I'll test it and if it works I'll open another PR just to update the README!

@soos3d soos3d closed this Aug 2, 2022
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.

3 participants