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.

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

Gave tags a default value for Decrement to match behaviour of Increment #60

Merged
merged 1 commit into from Nov 30, 2017
Merged

Gave tags a default value for Decrement to match behaviour of Increment #60

merged 1 commit into from Nov 30, 2017

Conversation

sqdk
Copy link

@sqdk sqdk commented Nov 22, 2017

No description provided.

@sqdk
Copy link
Author

sqdk commented Nov 22, 2017

I was wrapping this in F# and stumbled upon tags not having a default value for Decrement. On all other functions it has a default value of null.

Not sure if there is a good reason for this, or if its a breaking change that needs a major bump.

@yannmh
Copy link
Member

yannmh commented Nov 30, 2017

Thanks for fixing this @sqdk.

I am surprised this wasn't noticed before, I don't see any good reason for this.

@yannmh yannmh merged commit 42c71bd into DataDog:master Nov 30, 2017
@sqdk sqdk deleted the change-decrement-signature-to-match-increment branch December 7, 2017 11:37
@sqdk
Copy link
Author

sqdk commented Dec 12, 2017

Closes #61

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.

None yet

2 participants