Skip to content

Handle null and empty lines#8

Open
wallymathieu wants to merge 1 commit into
andyfeller:mainfrom
SolidifyDemo:null-handling
Open

Handle null and empty lines#8
wallymathieu wants to merge 1 commit into
andyfeller:mainfrom
SolidifyDemo:null-handling

Conversation

@wallymathieu
Copy link
Copy Markdown

No description provided.

@wallymathieu wallymathieu requested a review from andyfeller as a code owner May 8, 2023 12:56
@andyfeller
Copy link
Copy Markdown
Owner

Thanks for the contribution, @wallymathieu! 🎉 If I can ask for my own understanding, what kind of situation arose that required these changes being needed?

From all appearances this seems like a simple change that I'm all to happy to accept, I just want to consider whether any additional changes are necessary.

@wallymathieu
Copy link
Copy Markdown
Author

Mostly it was to try out this project. I got some errors and wanted to get fewer null reference exceptions while entering for instance invalid input.

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