Skip to content

fix: disable timeout on DbTool level#8357

Merged
Rjected merged 1 commit into
paradigmxyz:mainfrom
wlawt:issue-7707-dbtool
May 23, 2024
Merged

fix: disable timeout on DbTool level#8357
Rjected merged 1 commit into
paradigmxyz:mainfrom
wlawt:issue-7707-dbtool

Conversation

@wlawt

@wlawt wlawt commented May 22, 2024

Copy link
Copy Markdown
Contributor

Resolves #7707 .

Overview

This PR disables timeout on the DbTool level, as it's only used by the CLI.

Test Plan

  • Ran the unit tests
@wlawt wlawt requested a review from onbjerg as a code owner May 22, 2024 22:57
@wlawt

wlawt commented May 22, 2024

Copy link
Copy Markdown
Contributor Author

one issue i had while getting started was that in main and also my branch I was failing tests::etl_hashes. I'm not sure how to resolve this and test my changes fully

@wlawt wlawt force-pushed the issue-7707-dbtool branch 2 times, most recently from 9e3803b to 457c57b Compare May 22, 2024 23:30

@mattsse mattsse left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, pending @shekhirin

@mattsse mattsse added the A-db Related to the database label May 23, 2024
@wlawt

wlawt commented May 23, 2024

Copy link
Copy Markdown
Contributor Author

not sure, but these lint issues don't look related to this PR? or is there a way for me to address these @mattsse @DaniPopes @shekhirin . thank u!

@DaniPopes

Copy link
Copy Markdown
Contributor

this is fixed on main, just rebase

@wlawt wlawt force-pushed the issue-7707-dbtool branch from 457c57b to 36c5bf1 Compare May 23, 2024 14:55
@wlawt

wlawt commented May 23, 2024

Copy link
Copy Markdown
Contributor Author

done @DaniPopes , thank u!

@Rjected Rjected left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Rjected Rjected added this pull request to the merge queue May 23, 2024
Merged via the queue into paradigmxyz:main with commit 62e05b5 May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-db Related to the database

4 participants