Skip to content

Conversation

scout208
Copy link
Contributor

The first implementation of RangeTest should used exptected/1000 instead of ToleranceHelper because ToleranceHelper is not defined until the refactor so it would not build.

The first implementation of `RangeTest` should used `exptected/1000` instead of `ToleranceHelper` because `ToleranceHelper` is not defined until the refactor so it would not build.
@PRMerger20
Copy link
Contributor

@scout208 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Jak-MS Jak-MS added the aq-pr-triaged tracking label for the PR review team label Sep 10, 2019
Copy link
Member

@gewarren gewarren left a comment

Choose a reason for hiding this comment

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

@scout208 Thanks for fixing this code.

@gewarren gewarren merged commit 1a28fac into MicrosoftDocs:master Sep 27, 2019
@scout208 scout208 deleted the patch-3 branch September 27, 2019 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aq-pr-triaged tracking label for the PR review team change sent to author do-not-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants