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

initial code review #1

Closed
wants to merge 37 commits into from
Closed

initial code review #1

wants to merge 37 commits into from

Conversation

joe-p
Copy link
Collaborator

@joe-p joe-p commented Aug 23, 2024 •
edited
Loading

This PR contains all of the initial code for the python subscriber library. The tests are 1:1 from the TS library.

@joe-p
more progress
f97c93a
@joe-p
wait for indexer to catchup
41ecec3
@joe-p
fix indexer filters and log encoding/matching
c9655cf
@joe-p
fix indexer balance change consolidation
02d8852
@joe-p
remove debugging prints
5f50f44
@joe-p
balance changes fixes and tests
fcc3631
@joe-p
remove old TODO
c7df3d6
@joe-p
subscribe_indexer in test_events
18ba66a
@joe-p
complex txns snapshots
75ea933
@joe-p
remove TODO
3a6d01f
@joe-p
stprf and keyreg snapshot tests
bb11cf9
@joe-p
remove print
e2f8cd7
@joe-p
test_on_error
3bbcd40
@joe-p
logging.basicConfig(level=logging.INFO)
65456c9
@joe-p
add markdown docs + some docstrings
c6f7b9f
@joe-p
add additional comments to examples
46b994d
@joe-p
preserve order of filters
5a87ecc
@joe-p
ruff and black
c3e1567
@joe-p
mypy
de26a0e
@joe-p
add workflows (except docs)
46e37f4
@joe-p
reST docstrings
1652fd4
@joe-p
sync_behaviour docstring
55c5dd3
@joe-p
remove groups from poetry
d5b4fbd
@joe-p
add pre-commit to dev deps
75e6675
@joe-p
add black as dev dep
ff1bf2d
@joe-p
run black
51da047
@joe-p
remove commented out typescript
c19d1e3
@joe-p joe-p changed the base branch from main to beta September 19, 2024 16:15
@joe-p joe-p closed this Sep 24, 2024
@joe-p joe-p deleted the dev branch September 24, 2024 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@robdmoore robdmoore robdmoore 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
@joe-p @robdmoore @aorumbayev