Skip to content

Conversation

@maxwell-aiden
Copy link
Member

This is an automatic PR generated by the CI/CD pipeline. This will be automatically fast-forward merged if successful.

tegefaulkes and others added 2 commits January 30, 2025 14:35
chore: added ctx to most of the vault domain

feat: added cancellation for most `polykey vaults` commands

chore: added ctx to most of `secrets` commands

chore: cleaned up `VaultInternal.ts`

chore: added ctx to `VaultInternal` and `VaultManager`

feat: added context cancellation to the entire `vault` domain

chore: cleaned up vaults handlers

chore: updated `meta` type to match signature

chore: made `SuccessOrErrorMessage` consistently tagged

chore: added test for cancellation to `VaultsSecretsRemove`

chore: working on dynamic handler cancellation

chore: added cancellation tests for all secrets commands

chore: removed cancellation from synchronous loops

chore: added tests for vaults handlers

feat: added cancellation and tests for some git operations

chore: updated arbitraries
…ctor-rpc-handlers-to-handle-cancellation

Add cancellation to RPC handlers for the `vaults` domain
@maxwell-aiden maxwell-aiden self-assigned this Jan 30, 2025
@maxwell-aiden
Copy link
Member Author

Pipeline Attempt on 13045312974 for 23176b7

https://github.com/MatrixAI/Polykey/actions/runs/13045312974

@maxwell-aiden
Copy link
Member Author

Pipeline Succeeded on 13045312974 for 23176b7

https://github.com/MatrixAI/Polykey/actions/runs/13045312974

@maxwell-aiden maxwell-aiden merged commit 23176b7 into master Jan 30, 2025
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants