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

set eventStore variable in the inner check #384

Merged
merged 1 commit into from
Mar 9, 2022
Merged

set eventStore variable in the inner check #384

merged 1 commit into from
Mar 9, 2022

Conversation

qzio
Copy link
Contributor

@qzio qzio commented Mar 9, 2022

Description

Avoid the eventStore variable to potentially be nil.

Affected Components

  • namespace/Event Store

Related Issues

Solution and Design

set the eventStore variable as part of the inner check.

Steps to test and verify

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.08%) to 68.724% when pulling 622561d on qzio:potential-namespace-eventstore-nil-error into 1a72481 on looplab:main.

@maxekman maxekman merged commit 7aebfcb into looplab:main Mar 9, 2022
@qzio qzio deleted the potential-namespace-eventstore-nil-error branch March 10, 2022 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maxekman maxekman maxekman approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@qzio @coveralls @maxekman