flaviostutz / flaviostutz/cdk-practical-constructs
addLogGroupForTracing should be only adding things on top a pre-existing deployOptions (check discussion on #48)
Open
enhancement
- Dominant language
- TypeScript
- Stars
- 7
- Forks
- 7
- PR merge metrics
- No merged PRs in 30d
Description
## Problem being solved
addLogGroupForTracing should be only adding things on top a pre-existing deployOptions
Check discussion on https://github.com/flaviostutz/cdk-practical-constructs/pull/48
## Proposal
Re-factor to make this more sane
Contributor guide
Research direction
Start by reading the implementation of addLogGroupForTracing and the discussion on pull request #48; the issue does not name a file or test. Trace how deployOptions is handled, then verify that pre-existing options are preserved while tracing-related settings are added.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- aws, typescript
- Domain
- cloud
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100