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 return to load() when missing arguments #61

Merged
merged 1 commit into from
Nov 3, 2021
Merged

Add return to load() when missing arguments #61

merged 1 commit into from
Nov 3, 2021

Conversation

weefuzzy
Copy link
Member

fixes #60 by adding missing return to exit on error

@weefuzzy weefuzzy added the bug Something isn't working label Oct 27, 2021
@weefuzzy weefuzzy added this to the beta 4 milestone Oct 27, 2021
@weefuzzy weefuzzy requested a review from jamesb93 October 27, 2021 12:17
@weefuzzy weefuzzy self-assigned this Oct 27, 2021
@tremblap tremblap self-requested a review October 28, 2021 08:59
@tremblap
Copy link
Member

Works fine here - I refrained from merging as @jamesb93 needs to test.

@tremblap tremblap merged commit ed6d074 into flucoma:main Nov 3, 2021
@weefuzzy weefuzzy deleted the fix/load-crash-no-args branch February 20, 2023 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jamesb93 jamesb93 jamesb93 approved these changes

@tremblap tremblap Awaiting requested review from tremblap

Assignees

@weefuzzy weefuzzy

Labels
bug Something isn't working
Projects
None yet
Milestone
beta 4
Development

Successfully merging this pull request may close these issues.

Crash calling load without arguments
3 participants
@weefuzzy @tremblap @jamesb93