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.

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

chore: Clarify resolution detail reason. #239

Merged
merged 4 commits into from
Feb 5, 2024
Merged

Conversation

kinyoklion
Copy link
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>
@kinyoklion kinyoklion marked this pull request as ready for review January 29, 2024 21:56
Signed-off-by: Ryan Lamb <4955475+kinyoklion@users.noreply.github.com>
specification/types.md Outdated Show resolved Hide resolved
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
6 checks passed
@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