Skip to content

Fix CLI nil reference#237

Merged
alexshtin merged 3 commits intotemporalio:masterfrom
alexshtin:fix/cli-nil-reference-2
Mar 20, 2020
Merged

Fix CLI nil reference#237
alexshtin merged 3 commits intotemporalio:masterfrom
alexshtin:fix/cli-nil-reference-2

Conversation

@alexshtin
Copy link
Copy Markdown
Contributor

What changed?
IndexedFields field was not properly initialized. Added initialization.

Why?
It caused nil reference in CLI.

How did you test it?
Run broken command locally.

Potential risks
No risk.

@alexshtin alexshtin requested a review from samarabbas March 20, 2020 19:39
@alexshtin alexshtin changed the title Fix/cli nil reference 2 Fix CLI nil reference Mar 20, 2020
@alexshtin alexshtin merged commit 6546ca0 into temporalio:master Mar 20, 2020
@alexshtin alexshtin deleted the fix/cli-nil-reference-2 branch March 20, 2020 21:32
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