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

Switch To Circle CI #57

Merged
merged 8 commits into from
Mar 27, 2020
Merged

Switch To Circle CI #57

merged 8 commits into from
Mar 27, 2020

Conversation

hannahhoward
Copy link
Collaborator

Goals

Graphsync should be on the same testing regime as go-bitswap

Implementation

  • Update modules (turns out FC ecosytem already switched to new datastore)
  • Switch to CircleCI using go-bitswap config
  • Fix lint errors (mostly mirroring go-bitswap fixes)
  • Update tests which fail on circle due to timing -- there are intentional context timeouts to prevent failing tests from locking, but they are set for super small timeouts which prevent the suite from running.

Use configs from bitswap
Since we are not relying on context cancellation as anything but a test failsafe, relax timings
significantly for slower machines
Since we are not relying on context cancellation as anything but a test failsafe, relax timings
@hannahhoward hannahhoward merged commit bd3f4b4 into master Mar 27, 2020
@aschmahmann aschmahmann mentioned this pull request Sep 22, 2020
72 tasks
@mvdan mvdan deleted the fix/switch-to-cirlce branch December 15, 2021 14:15
marten-seemann pushed a commit that referenced this pull request Mar 2, 2023
* feat(graphsync): setup alternate stores

add a method to allow swapping of an alternate store for a given request

* refactor(datatransfer): move all interfaces to top

move interfaces to the top level package to avoid circular imports

* feat(impl): support transport configurer

Support a per voucher type transport configurer to allow custom configuration of a transport

* fix(lint): fix lint/mod-tidy

* fix(impl): fix integration test
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.

1 participant