Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Provide correct exception message #580

Merged
merged 1 commit into from
Oct 4, 2018
Merged

Conversation

SeanFeldman
Copy link
Collaborator

Fixes #579
For an empty list do not throw argument null exception

@SeanFeldman SeanFeldman requested a review from a team as a code owner October 3, 2018 23:00
@nemakam nemakam added this to the 3.1.2 milestone Oct 3, 2018
@nemakam nemakam merged commit 93ab7b0 into Azure:dev Oct 4, 2018
@@ -159,6 +159,9 @@
<data name="TokenMissingExpiresOn" xml:space="preserve">
<value>The provided token does not specify the 'ExpiresOn' value.</value>
</data>
<data name="ListOfLockTokensCannotBeEmty" xml:space="preserve">
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit - please fix typo ListOfLockTokensCannotBeEmpty in the next commit

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@axisc I'm blind as a 🦇 - what typo?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ListOfLockTokensCannotBeEmty > ListOfLockTokensCannotBeEmpty

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @stjepan. Blind like a bat I am 😂

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR #583 raised

@SeanFeldman SeanFeldman deleted the issue-579 branch October 4, 2018 08:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants