Skip to content

refactor: deallocation try catch to call#1696

Merged
elhajin merged 1 commit intorelease-dev/duration-vaultsfrom
mike/deallocation-call
Jan 12, 2026
Merged

refactor: deallocation try catch to call#1696
elhajin merged 1 commit intorelease-dev/duration-vaultsfrom
mike/deallocation-call

Conversation

@eigenmikem
Copy link
Contributor

Motivation:

Explain here the context, and why you're making that change. What is the problem you're trying to solve.

Modifications:

Describe the modifications you've done.

Result:

After your change, what will change.

@elhajin elhajin merged commit f036c62 into release-dev/duration-vaults Jan 12, 2026
9 checks passed
@elhajin elhajin deleted the mike/deallocation-call branch January 12, 2026 13:15
eigenmikem added a commit that referenced this pull request Feb 13, 2026
<!-- 
    🚨 ATTENTION! 🚨 
    
This PR template is REQUIRED. PRs not following this format will be
closed without review.
    
    Requirements:
- PR title must follow commit conventions:
https://www.conventionalcommits.org/en/v1.0.0/
- Label your PR with the correct type (e.g., 🐛 Bug, ✨ Enhancement, 🧪
Test, etc.)
    - Provide clear and specific details in each section
-->

**Motivation:**

use call instead of try/catch

**Modifications:**

*Describe the modifications you've done.*

**Result:**

*After your change, what will change.*
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