Skip to content

Added System namespace to BankAccountTests.cs #3362

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

Closed
wants to merge 2 commits into from
Closed

Added System namespace to BankAccountTests.cs #3362

wants to merge 2 commits into from

Conversation

mavaddat
Copy link
Contributor

@mavaddat mavaddat commented Jun 2, 2019

The BankAccountTests.cs needs the System namespace for the ArgumentOutOfRangeException class that is used in Create and run new test methods section.

Before creating my pull request, I have checked my content against these quality criteria:

  • I did not choose the title in the metadata section and the H1 heading, so I did not consider search engine optimization (SEO)
  • This is not a new article, hence I did not add it to the table of contents
  • I did not update the "ms.date" metadata, since this is not a new or significantly updated article
  • I have introduced and explained technical terms and concepts, and I have spelled out acronyms on first mention
  • I defer to the page creators whether this this page should be linked to from other pages or Microsoft web sites

For more information about creating content for docs.microsoft.com, see the contributor guide at https://docs.microsoft.com/contribute/.

The BankAccountTests.cs needs the `System` namespace for the `ArgumentOutOfRangeException` class that is used in [Create and run new test methods](#create-and-run-new-test-methods) section.
@PRMerger9
Copy link
Contributor

@mavaddat : Thanks for your contribution! The author, @gewarren, has been notified to review your proposed change.

@PRMerger20
Copy link
Contributor

@jborsecnik : Thanks for your contribution! The author, @gewarren, has been notified to review your proposed change.

Copy link
Contributor Author

@mavaddat mavaddat left a comment

Choose a reason for hiding this comment

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

Looks good. I forgot the indefinite article ('a') prior to the namespace, so I was hoping someone would throw that in. Thanks!

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

Successfully merging this pull request may close these issues.

5 participants