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

Verbose errors during interpretation #46

Open
namnc opened this issue Apr 5, 2024 · 3 comments · May be fixed by #74
Open

Verbose errors during interpretation #46

namnc opened this issue Apr 5, 2024 · 3 comments · May be fixed by #74
Assignees
Milestone

Comments

@namnc
Copy link
Owner

namnc commented Apr 5, 2024

Error is very generic right now it is not super helpful, can we improve the information display upon error?

@namnc
Copy link
Owner Author

namnc commented May 8, 2024

We just need very basic information for 1st release.

@namnc namnc removed the Release 0.1 label May 8, 2024
@namnc namnc added this to the Release 0.1 milestone May 8, 2024
@namnc namnc added the good first issue Good for newcomers label May 15, 2024
@namnc
Copy link
Owner Author

namnc commented Jun 13, 2024

@curryrasul I think what this needs to handle are:

pub enum RuntimeError {
#[error("Access Error")]
AccessError,
#[error("Error retrieving context")]
ContextRetrievalError,
#[error("Empty context stack")]
EmptyContextStack,
#[error("Index out of bounds")]
IndexOutOfBounds,
#[error("Item already declared")]
ItemAlreadyDeclared,
#[error("Item not declared: {0}")]
ItemNotDeclared(String),
#[error("No context to inherit from")]
NoContextToInheritFrom,
#[error("Data Item content is not a single value")]
NotAValue,
#[error("Unsupported data type")]
UnsupportedDataType,
}

@namnc
Copy link
Owner Author

namnc commented Jun 13, 2024

In particular I usually ran into Error: RuntimeError(IndexOutOfBounds) and not able to tell which index access cause this and have to do debugging ...

@curryrasul curryrasul linked a pull request Jun 16, 2024 that will close this issue
@curryrasul curryrasul removed the good first issue Good for newcomers label Jun 28, 2024
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 a pull request may close this issue.

2 participants