Skip to content

Closes #352#354

Open
only1rezie wants to merge 2 commits into
Toolbox-Lab:mainfrom
only1rezie:patch-1
Open

Closes #352#354
only1rezie wants to merge 2 commits into
Toolbox-Lab:mainfrom
only1rezie:patch-1

Conversation

@only1rezie

Copy link
Copy Markdown

git add .
git commit -m "Implement concise error summary API"
git push origin my-feature

@codeZe-us

Copy link
Copy Markdown
Contributor

@only1rezie i can't see any code in this PR

@only1rezie

Copy link
Copy Markdown
Author

Hello! I noticed HostError already implements a summary() method in host_error.rs. Before I continue, should this issue update the existing implementation to the new 80-character Tier 1 format, or is there another summary() implementation you intended me to modify?

@only1rezie

Copy link
Copy Markdown
Author

Thanks for confirming. Before I update summary(), should the implementation simply truncate the existing summaries to the new 80-character Tier 1 format, or would you prefer introducing dedicated short summaries (e.g. [BUDGET] CPU Exceeded) instead of truncating the existing descriptive text?

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.

2 participants