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

Make object instances in tests readonly #1436

Merged

ob-stripe merged 1 commit into integration-v23 from ob-readonly

Dec 14, 2018

Contributor

ob-stripe commented Dec 14, 2018

It's good practice to declare fields that are only assigned in constuctors as readonly.


          Make object instances in tests readonly

a3de929

ob-stripe mentioned this pull request

stripe-dotnet v22.0.0 #1416

Merged

29 tasks

Contributor Author

ob-stripe commented Dec 14, 2018

Tests still pass, and only test files were modified in this PR, so going to self-approve.

ob-stripe added the approved label

ob-stripe self-assigned this

ob-stripe merged commit 4f0d5cf into integration-v23

ob-stripe deleted the ob-readonly branch

December 14, 2018 22:38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels