Skip to content

test(calculate): verify streak formulas for massive single-day commit spike timeline#1853

Merged
JhaSourav07 merged 3 commits into
JhaSourav07:mainfrom
Vishvadharman:test-calculate-commit-spike
May 31, 2026
Merged

test(calculate): verify streak formulas for massive single-day commit spike timeline#1853
JhaSourav07 merged 3 commits into
JhaSourav07:mainfrom
Vishvadharman:test-calculate-commit-spike

Conversation

@Vishvadharman
Copy link
Copy Markdown
Contributor

Description

Fixes #1507

Added a new test case in lib/calculate.test.ts to verify streak calculations for a timeline containing a massive single-day commit spike alongside low-frequency contribution periods.

This ensures streak values are calculated based on active contribution days, not the raw number of commits on a single day.

Pillar

  • 🎨 Pillar 1 — New Theme Design
  • 📐 Pillar 2 — Geometric SVG Improvement
  • 🕐 Pillar 3 — Timezone Logic Optimization
  • 🛠️ Other (Bug fix, refactoring, docs)

Visual Preview

Not applicable — this PR only adds test coverage.

Checklist before requesting a review:

  • I have read the CONTRIBUTING.md file.
  • I have tested these changes locally.
  • I have run npm run format and npm run lint locally and resolved all errors.
  • My commits follow the Conventional Commits format.
  • I have updated README.md if I added a new theme or URL parameter.
  • I have started the repo.
  • I have made sure that i have only one commit to merge in this PR.
  • The SVG output matches the CommitPulse "premium quality" aesthetic standard.
  • (Recommended) I joined the CommitPulse Discord community for contributor discussions, mentorship, and faster PR support.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 30, 2026

@Vishvadharman is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the type:testing Adding, updating, or fixing tests label May 30, 2026
@JhaSourav07
Copy link
Copy Markdown
Owner

resolve the conflicts please

@JhaSourav07 JhaSourav07 added the needs-rebase This PR has merge conflicts and needs a rebase. label May 30, 2026
Copy link
Copy Markdown
Collaborator

@Aamod007 Aamod007 left a comment

Choose a reason for hiding this comment

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

This PR has the
eeds-rebase\ label. Please rebase and re-request review.

Copy link
Copy Markdown
Collaborator

@Aamod007 Aamod007 left a comment

Choose a reason for hiding this comment

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

Unapproving — this PR has a
eeds-rebase\ or \status:blocked\ label. Please resolve that first and re-request review.

@github-actions github-actions Bot removed the needs-rebase This PR has merge conflicts and needs a rebase. label May 30, 2026
@JhaSourav07 JhaSourav07 added the status: blocked PRs with merge conflicts, failing builds, label May 30, 2026
@Vishvadharman Vishvadharman deleted the test-calculate-commit-spike branch May 31, 2026 06:27
@Vishvadharman Vishvadharman restored the test-calculate-commit-spike branch May 31, 2026 06:27
@Vishvadharman Vishvadharman reopened this May 31, 2026
@JhaSourav07 JhaSourav07 merged commit a14d064 into JhaSourav07:main May 31, 2026
9 of 10 checks passed
@JhaSourav07 JhaSourav07 added GSSoC 2026 level:beginner Small changes Usually isolated fixes or simple UI/text updates. quality:exceptional Outstanding contribution with exceptional implementation quality, testing. gssoc:approved PR has been reviewed and accepted for valid contribution points mentor:Aamod007 labels May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved PR has been reviewed and accepted for valid contribution points GSSoC 2026 level:beginner Small changes Usually isolated fixes or simple UI/text updates. mentor:Aamod007 quality:exceptional Outstanding contribution with exceptional implementation quality, testing. status: blocked PRs with merge conflicts, failing builds, type:testing Adding, updating, or fixing tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test(calculate): verify streak formulas for massive single-day commit spike timeline (Variation 4)

3 participants