Skip to content

chore: Clarify resolution detail reason.#239

Merged
toddbaert merged 4 commits into
mainfrom
rlamb/clarify-reasons
Feb 5, 2024
Merged

chore: Clarify resolution detail reason.#239
toddbaert merged 4 commits into
mainfrom
rlamb/clarify-reasons

Conversation

@kinyoklion

Copy link
Copy Markdown
Member

This PR

Attempt to clarify the reason field of the resolution details.

Related Issues

Fixes #236

Notes

Follow-up Tasks

How to test

Signed-off-by: Ryan Lamb <4955475+kinyoklion@users.noreply.github.com>
Comment thread specification/types.md
@kinyoklion kinyoklion marked this pull request as ready for review January 29, 2024 21:56
@kinyoklion kinyoklion requested a review from toddbaert as a code owner January 29, 2024 21:56
Signed-off-by: Ryan Lamb <4955475+kinyoklion@users.noreply.github.com>
Comment thread specification/types.md
Comment thread specification/types.md Outdated
Co-authored-by: Michael Beemer <beeme1mr@users.noreply.github.com>
Signed-off-by: Ryan Lamb <4955475+kinyoklion@users.noreply.github.com>
@toddbaert toddbaert merged commit 738bb41 into main Feb 5, 2024
@toddbaert toddbaert deleted the rlamb/clarify-reasons branch February 5, 2024 17:29
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.

Clarification of reason for evaluation detail and evaluation.

4 participants