fix bug in new_indent_parser - Githubissues

nlmatics / nlm-ingestor

This repo provides the server side code for llmsherpa API to connect. It includes parsers for various file formats.

https://www.nlmatics.com

Apache License 2.0

1.11k stars 160 forks source link

fix bug in new_indent_parser #87

Closed hguhlich closed 1 month ago

hguhlich commented 3 months ago

If curr_header is not set first, this leads to a type error, see #21

Description of the change

Check if curr_header is initialized in the same way as in the if statement above.

Type of change

[X] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues

Fix [#21]()

Checklists

Development

[ ] Lint rules pass locally
[ ] The code changed/added as part of this pull request has been covered with tests
[ ] All tests related to the changed code pass in development

Code review

[ ] This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] "Ready for review" label attached to the PR and reviewers mentioned in a comment
[x] Changes have been reviewed by at least one other engineer
[ ] Issue from task tracker has a link to this pull request