Skip to content
This repository has been archived by the owner on Aug 19, 2024. It is now read-only.

Use new get_task functionality in tests #2

Merged
merged 2 commits into from
Apr 11, 2024
Merged

Conversation

qstokkink
Copy link
Owner

@qstokkink qstokkink commented Apr 11, 2024

This PR:

  • Updates the IPv8 pointer.
  • Updates the tests to use get_task functionality.
  • Updates the Python versions for Linux and Mac Actions.

Failures:

  • Linux: ipv8-rust-tunnels not available on Python 3.7.
  • Mac: libtorrent not available on Python 3.12.
  • Validate:

set_pending_status
Github returned error "Resource not accessible by integration - https://docs.github.com/rest/commits/statuses#create-a-commit-status" when setting status on commit: 2bc1c2e9c1d60aeddfaf30f1bb5b52de4a59bfeb
Request object:
{
 "context": "Cross-env Validation",
 "description": "Pending..",
 "state": "pending",
 "owner": "qstokkink",
 "repo": "TriblerExperimental",
 "sha": "2bc1c2e9c1d60aeddfaf30f1bb5b52de4a59bfeb",
 "target_url": "https://github.com/qstokkink/TriblerExperimental/actions/runs/8647285243"
} Possible issues could be that the token used does not have access to the repository containing the commit or the commit/repository does not exist.

@qstokkink
Copy link
Owner Author

validate

@qstokkink qstokkink marked this pull request as ready for review April 11, 2024 13:19
@qstokkink qstokkink changed the title WIP: Use new get_task functionality in tests READY: Use new get_task functionality in tests Apr 11, 2024
@qstokkink
Copy link
Owner Author

I'll fix the validate in another PR.

@qstokkink qstokkink requested a review from egbertbouman April 11, 2024 13:21
@qstokkink qstokkink changed the title READY: Use new get_task functionality in tests Use new get_task functionality in tests Apr 11, 2024
@qstokkink qstokkink merged commit 99d7e87 into highlander Apr 11, 2024
3 of 4 checks passed
@qstokkink qstokkink deleted the upd_get_task branch April 11, 2024 14:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants