Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Feature]: Split AssertLib Functionality From DebugLib #559

Open
os-d opened this issue Sep 11, 2023 · 0 comments
Open

[Feature]: Split AssertLib Functionality From DebugLib #559

os-d opened this issue Sep 11, 2023 · 0 comments
Assignees
@os-d
Labels
impact:breaking-change Requires integration attention state:backlog In the backlog state:needs-triage Needs to triaged to determine next steps type:feature-request A new feature proposal urgency:medium Important with a moderate impact

Comments

@os-d
Copy link
Contributor

os-d commented Sep 11, 2023

Feature Overview

Today, assert functionality lives inside of DebugLib, which is mixing two separate features into one library and requires every DebugLib implementation to change in response to changes to assert functionality.

Solution Overview

A cleaner and more maintainable architecture would be to split the assert functionality into an AssertLib with a well defined interface that can be consumed by DebugLib.

Alternatives Considered

No response

Urgency

Medium

Are you going to implement the feature request?

I will implement the feature

Do you need maintainer feedback?

No maintainer feedback needed

Anything else?

No response
@os-d os-d added impact:breaking-change Requires integration attention state:backlog In the backlog state:needs-triage Needs to triaged to determine next steps type:feature-request A new feature proposal labels Sep 11, 2023
@os-d os-d self-assigned this Sep 11, 2023
@github-actions github-actions bot added the urgency:medium Important with a moderate impact label Sep 11, 2023
@os-d os-d mentioned this issue Sep 11, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@os-d os-d

Labels
impact:breaking-change Requires integration attention state:backlog In the backlog state:needs-triage Needs to triaged to determine next steps type:feature-request A new feature proposal urgency:medium Important with a moderate impact
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@os-d