Switch to `try` / `finally` for wrapper cleanup
Open
enhancement
good first issue
- Dominant language
- JavaScript
- Stars
- 226
- Forks
- 14
- PR merge metrics
- No merged PRs in 30d
Description
See suggestion on https://github.com/azavea/loam/pull/112 -- this dramatically simplifies cleanup code. The wrappers from that PR already use this format; this issue is to migrate all the preexisting wrappers to the same style.
Contributor guide
No contributing guide indexed for this repository
Research direction
Read the suggestion on pull request 112 and compare its wrapper cleanup style with the preexisting wrappers. Migrate all older wrappers to the same try/finally format, then verify that cleanup behavior remains intact across those wrappers.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- web-dev
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100