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

Improve missing enumeration value error message #4339

Merged

Conversation

davisp
Copy link
Contributor

@davisp davisp commented Sep 8, 2023

Previously, the error message was a cryptic message about a failed integral cast which is terrible for user experience. This updates things to make the error message specifically state that the enumeration value wasn't found.


TYPE: IMPROVEMENT
DESC: Fix enumeration value not found error message

@shortcut-integration
Copy link

@davisp davisp requested a review from johnkerl September 8, 2023 20:07
Previously, the error message was a cryptic message about a failed
integral cast which is terrible for user experience. This updates things
to make the error message specifically state that the enumeration value
wasn't found.
@davisp davisp force-pushed the pd/sc-33895/fix-enumeration-value-not-found-error-message branch from 37d4036 to c35ce63 Compare September 8, 2023 21:06
@ihnorton ihnorton merged commit 79a5406 into dev Sep 9, 2023
55 checks passed
@ihnorton ihnorton deleted the pd/sc-33895/fix-enumeration-value-not-found-error-message branch September 9, 2023 19:02
github-actions bot pushed a commit that referenced this pull request Sep 9, 2023
Previously, the error message was a cryptic message about a failed integral cast which is terrible for user experience. This updates things to make the error message specifically state that the enumeration value wasn't found.

---
TYPE: IMPROVEMENT
DESC: Fix enumeration value not found error message

(cherry picked from commit 79a5406)
ihnorton pushed a commit that referenced this pull request Sep 11, 2023
Previously, the error message was a cryptic message about a failed integral cast which is terrible for user experience. This updates things to make the error message specifically state that the enumeration value wasn't found.

---
TYPE: IMPROVEMENT
DESC: Fix enumeration value not found error message

(cherry picked from commit 79a5406)

Co-authored-by: Paul J. Davis <paul.davis@tiledb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants