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

Update IPv8 pointer and tests #8346

Merged
merged 4 commits into from
Dec 16, 2024
Merged

Update IPv8 pointer and tests #8346

merged 4 commits into from
Dec 16, 2024

Conversation

qstokkink
Copy link
Contributor

@qstokkink qstokkink commented Dec 16, 2024 •
edited
Loading

This PR:

  • Updates IPv8 to the 3.0 version.
  • Updates the Tribler tests to use a single MockTriblerConfigManager class.
  • Updates the Tribler tests to use the IPv8 MockRequest class.
  • Updates the ruff configuration to use Python 3.9+ syntax.
  • Updates the ruff GitHub Action to ignore changed files in the pyipv8 submodule.

@qstokkink qstokkink added this to the 8.1.0 milestone Dec 16, 2024
@qstokkink qstokkink changed the title WIP: Update IPv8 pointer and tests ONHOLD: Update IPv8 pointer and tests Dec 16, 2024
@qstokkink
Copy link
Contributor Author

This is technically ready for review, but I won't merge it until after the 8.0.7 release.

@qstokkink qstokkink marked this pull request as ready for review December 16, 2024 12:14
@qstokkink qstokkink removed this from the 8.1.0 milestone Dec 16, 2024
@qstokkink qstokkink changed the title ONHOLD: Update IPv8 pointer and tests Update IPv8 pointer and tests Dec 16, 2024
@qstokkink qstokkink merged commit cf9ab15 into Tribler:main Dec 16, 2024
7 checks passed
@qstokkink qstokkink deleted the update_ipv8 branch December 16, 2024 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@egbertbouman egbertbouman egbertbouman approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@qstokkink @egbertbouman