Skip to content

Enhance error message clarity and context across error enums#380

Closed
GarmashAlex wants to merge 1 commit intonexus-xyz:mainfrom
GarmashAlex:hh12
Closed

Enhance error message clarity and context across error enums#380
GarmashAlex wants to merge 1 commit intonexus-xyz:mainfrom
GarmashAlex:hh12

Conversation

@GarmashAlex
Copy link
Contributor

Is this resolving a feature or a bug?

feature

Are there existing issue(s) that this PR would close?

no

Describe your changes.

  • Added specific details to vague errors like "Invalid memory layout"
  • Clarified errors related to instruction execution and registry
  • Added context about why certain operations failed

@github-actions
Copy link

github-actions bot commented Mar 29, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@GarmashAlex
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@sjudson
Copy link
Contributor

sjudson commented Apr 1, 2025

Thanks a lot for the PR. Ultimately these are minor aesthetic changes in some non-user facing code, and as it doesn't fix a concrete issue or contribute new functionality, I'm going to pass on it.

@sjudson sjudson closed this Apr 1, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Apr 1, 2025
@GarmashAlex GarmashAlex deleted the hh12 branch June 24, 2025 16:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants