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

Adding optional tests … #93

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ulrichard
Copy link

… to showcase that it currently only works with electrs servers

@ulrichard ulrichard force-pushed the feature/test_servers branch from 4b1bee4 to 2341936 Compare January 5, 2023 10:47
@notmandatory
Copy link
Member

notmandatory commented Nov 12, 2023

Hi, I'm just getting caught up on PRs in this repo. It's good to have a way to show which electrum servers this repo works and doesn't work with, but I'd rather not have a test that always fails. Also I also don't want to use non-local servers that could fail for other reasons. How would you feel about getting this to at least pass with electrsd and commenting out the other servers as work for future PRs when and if we can get those other servers working and locally testable?

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