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
feat(review): allow raw diff for reviewPullRequest #137
Conversation
Codecov Report
@@ Coverage Diff @@
## master #137 +/- ##
==========================================
- Coverage 86.34% 86.30% -0.04%
==========================================
Files 22 41 +19
Lines 2145 3986 +1841
Branches 150 296 +146
==========================================
+ Hits 1852 3440 +1588
- Misses 292 538 +246
- Partials 1 8 +7
Continue to review full report at Codecov.
|
}, | ||
]); | ||
|
||
const stubMakePr = proxyquire.noCallThru()( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like we probably should be able to use sinon, rather than proxyquire here? It feels a bit weird to me to have both approaches to mocking. I've been landing on a pattern where, if I'm tempted to use proxyquire
, I'll instead encapsulate behavior inside a helper method and mock that with sinon.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to consider my review blocking 👍
🤖 I have created a release \*beep\* \*boop\* --- ## [1.8.0](https://www.github.com/googleapis/code-suggester/compare/v1.7.0...v1.8.0) (2020-10-09) ### Features * **review:** allow raw diff for reviewPullRequest ([#137](https://www.github.com/googleapis/code-suggester/issues/137)) ([841526d](https://www.github.com/googleapis/code-suggester/commit/841526d720b6af4a21cf3ab08e5749a812fb5c30)) ### Bug Fixes * handle addition-only and deletion-only changes ([#144](https://www.github.com/googleapis/code-suggester/issues/144)) ([ab936a2](https://www.github.com/googleapis/code-suggester/commit/ab936a251171ca5aafc160f3506db54a7cb31e8a)), closes [#126](https://www.github.com/googleapis/code-suggester/issues/126) [#127](https://www.github.com/googleapis/code-suggester/issues/127) * the review comment should go on the old file lines, not the new file lines ([#140](https://www.github.com/googleapis/code-suggester/issues/140)) ([662391f](https://www.github.com/googleapis/code-suggester/commit/662391f2ae18e8ec997e0961430ce3ccc6f11cee)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please).
No description provided.