Skip to content

Conversation

@thelukewalton
Copy link
Collaborator

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

src/scripts/prevCoverage.ts:113

  • The command ‘git add . && git stash’ may stash unintended changes beyond the build artifacts. Consider narrowing the scope of the stashed changes (e.g., by specifying only the relevant paths) to avoid accidentally stashing unrelated work.
await exec("git add . && git stash");

src/scripts/prevCoverage.ts:134

  • If no changes were stashed or if the stash was already applied, ‘git stash pop’ might cause an error. Consider adding a check to verify that a stash exists or handling errors from this command.
await exec("git stash pop");

@thelukewalton thelukewalton merged commit 1f1dedb into main Apr 14, 2025
4 checks passed
@thelukewalton thelukewalton deleted the fix/builds branch April 14, 2025 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants