Skip to content
This repository was archived by the owner on Mar 6, 2024. It is now read-only.

Commit e6056fd

Browse files
authored
do not submit review when it's disabled (#434)
<!-- This is an auto-generated comment: release notes by OSS CodeRabbit --> ### Summary by CodeRabbit ## Release Notes - **New Feature**: Enhanced the review submission process in `src/review.ts`. The update includes a final summary comment before submitting the review, improving clarity and communication. > 🎉 Here's to the code that's ever so bright, > > A new line added, enhancing our might. > > With summaries clear, we take flight, > > In the realm of code, it's a delightful sight! 🚀 <!-- end of auto-generated comment: release notes by OSS CodeRabbit -->
1 parent a59e96c commit e6056fd

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

dist/index.js

+2-2
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/review.ts

+7-7
Original file line numberDiff line numberDiff line change
@@ -730,17 +730,17 @@ ${
730730
existingCommitIdsBlock,
731731
context.payload.pull_request.head.sha
732732
)}`
733+
734+
// post the review
735+
await commenter.submitReview(
736+
context.payload.pull_request.number,
737+
commits[commits.length - 1].sha,
738+
statusMsg
739+
)
733740
}
734741

735742
// post the final summary comment
736743
await commenter.comment(`${summarizeComment}`, SUMMARIZE_TAG, 'replace')
737-
738-
// post the review
739-
await commenter.submitReview(
740-
context.payload.pull_request.number,
741-
commits[commits.length - 1].sha,
742-
statusMsg
743-
)
744744
}
745745

746746
const splitPatch = (patch: string | null | undefined): string[] => {

0 commit comments

Comments
 (0)