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.

Sign up for GitHub

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

Jump to bottom

Restore Test Book #1523

Merged
merged 5 commits into from
Mar 16, 2024
Merged

Restore Test Book #1523

merged 5 commits into from
Mar 16, 2024

Conversation

ouziel-slama
Copy link
Contributor

No description provided.

github-advanced-security[bot]
github-advanced-security bot found potential problems Mar 16, 2024
View reviewed changes
counterparty-lib/counterpartylib/lib/backend/addrindexrs.py
return GET_OLDEST_TX_HARDCODED[hardcoded_key]
result = GET_OLDEST_TX_HARDCODED[hardcoded_key]
else:
global ADDRINDEXRS_CLIENT

Check warning

Code scanning / pylint

Using the global statement. Warning

Using the global statement.
@ouziel-slama
Copy link
Contributor Author

  1. mock addrindexrs when running book_test.py.
  2. reparse 5 checkpoints
  3. In github workflow: bootstrap testnet database then run book_test.py

@ouziel-slama ouziel-slama changed the title [WIP] Restore Test Book Restore Test Book Mar 16, 2024
@ouziel-slama ouziel-slama mentioned this pull request Mar 16, 2024
Closed
2 tasks
adamkrellenstein
adamkrellenstein approved these changes Mar 16, 2024
View reviewed changes
.github/workflows/test_book.yml Outdated Show resolved Hide resolved
.github/workflows/test_book.yml Show resolved Hide resolved
@ouziel-slama ouziel-slama merged commit ddd2143 into develop Mar 16, 2024
13 checks passed
@ouziel-slama ouziel-slama deleted the testbook branch March 16, 2024 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adamkrellenstein adamkrellenstein adamkrellenstein approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ouziel-slama @adamkrellenstein @Ouziel