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.

Sign up for GitHub

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

Jump to bottom

Add context in Import from toml #355

Open
vickysharma0812 opened this issue Dec 26, 2023 · 0 comments
Open

Add context in Import from toml #355

vickysharma0812 opened this issue Dec 26, 2023 · 0 comments
Assignees
@vickysharma0812 @shishiousan
Labels
enhancement New feature or request toml
Milestone
v24.04.00

Comments

@vickysharma0812
Copy link
Member

Enhancement

Current importFromToml methods does not contain context argument. See this example: https://toml-f.readthedocs.io/en/latest/how-to/error.html

We should add context as optional argument.

When context is present, then we use context to print the error. Otherwise, we use normal display method to print the error.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vickysharma0812 vickysharma0812

@shishiousan shishiousan

Labels
enhancement New feature or request toml
Projects
None yet
Milestone
v24.04.00
Development

No branches or pull requests
2 participants
@vickysharma0812 @shishiousan