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

feat: make anchoring fast by default #191

Merged
merged 2 commits into from
Jun 25, 2024
Merged

feat: make anchoring fast by default #191

merged 2 commits into from
Jun 25, 2024

Conversation

nathanielc
Copy link
Collaborator

When keramik is used to deploy CAS it is only in the context of a hermetic testing env. Therefore we can assume that anchoring should be as fast as possible in order to all for faster tests etc.

Makes two changes:

  1. Reduces worker delay to 1s instead of 1m after completing a batch.
  2. Configures ganache to instantly mine any transaction.

When keramik is used to deploy CAS it is only in the context of a
hermetic testing env. Therefore we can assume that anchoring should be
as fast as possible in order to all for faster tests etc.

Makes two changes:

1. Reduces worker delay to 1s instead of 1m after completing a batch.
2. Configures ganache to instantly mine any transaction.
@nathanielc nathanielc requested a review from smrz2001 June 25, 2024 15:15
Copy link
Collaborator

@smrz2001 smrz2001 left a comment

Choose a reason for hiding this comment

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

🚀

@nathanielc nathanielc enabled auto-merge June 25, 2024 15:36
@nathanielc nathanielc added this pull request to the merge queue Jun 25, 2024
Merged via the queue into main with commit c8f34aa Jun 25, 2024
4 of 5 checks passed
samika98 pushed a commit that referenced this pull request Jul 12, 2024
* feat: make anchoring fast by default

When keramik is used to deploy CAS it is only in the context of a
hermetic testing env. Therefore we can assume that anchoring should be
as fast as possible in order to all for faster tests etc.

Makes two changes:

1. Reduces worker delay to 1s instead of 1m after completing a batch.
2. Configures ganache to instantly mine any transaction.

* tests
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.

2 participants