Skip to content

check for null config too#418

Merged
boulter merged 1 commit intomasterfrom
check-for-null-config
Apr 1, 2020
Merged

check for null config too#418
boulter merged 1 commit intomasterfrom
check-for-null-config

Conversation

@boulter
Copy link
Contributor

@boulter boulter commented Apr 1, 2020

The check added in #410 checked for a null interpreter, but not a null config, a situation that exists in some of our unit tests.

@gabru-md

@boulter boulter merged commit 0f50fee into master Apr 1, 2020
@boulter boulter deleted the check-for-null-config branch April 1, 2020 18:00
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.

1 participant