Skip to content
This repository has been archived by the owner on Jan 3, 2025. It is now read-only.

feat: day 25 #30

Merged
merged 3 commits into from
Dec 25, 2024
Merged

feat: day 25 #30

merged 3 commits into from
Dec 25, 2024

Conversation

rnbguy
Copy link
Owner

@rnbguy rnbguy commented Dec 25, 2024

@rnbguy rnbguy requested a review from Copilot December 25, 2024 11:07
Copilot
Copilot AI reviewed Dec 25, 2024
View reviewed changes
Copy link
Contributor

@Copilot 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 4 changed files in this pull request and generated 1 comment.

Files not reviewed (2)
  • day25/deno.json: Language not supported
  • day25/example.txt: Language not supported
Comments suppressed due to low confidence (1)

day25/test.ts:4

  • Consider adding direct tests for the doesFit method to improve test coverage.
Deno.test(async function testExample() {

day25/mod.ts Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 25, 2024 •
edited
Loading

LCOV of commit dfb1002 during CI #68

Summary coverage rate:
  lines......: 83.9% (2089 of 2491 lines)
  functions..: 96.2% (176 of 183 functions)
  branches...: 88.3% (476 of 539 branches)

Files changed coverage rate: n/a

@rnbguy rnbguy merged commit 0a69387 into main Dec 25, 2024
3 checks passed
@rnbguy rnbguy deleted the day25 branch December 25, 2024 11:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

Copilot code review Copilot Copilot left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rnbguy