Comcast / Comcast/mamba

improve (HLS)ValidationIssue

Open
#78 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Swift
Stars
191
Forks
43
PR merge metrics
No merged PRs in 30d

Description

### Description

comment I ran across in our current implementation:
// mamba validation issues have a poor interface. Best I can do is search in the description field when trying to find a specific issue in a list.

### Tasks

- [ ] change `public init(description: IssueDescription, severity: IssueSeverity)` to store the `IssueDescription` instead of just the description string
- [ ] change ` public init(description: String, severity: IssueSeverity)` to store the description as a `.undefined(String)` within `IssueDescription` enum
- [ ] change `description` attribute to an accessor of the string of the stored `IssueDescription`

### Open Questions

- Does this seem like the right approach IYO @dcoufal ?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.