Refactor Calculate Module: Document DaysInYear Export for Reuse#8353
Refactor Calculate Module: Document DaysInYear Export for Reuse#8353tmdeveloper007 wants to merge 1 commit into
Conversation
|
This pull request improves the documentation of the daysInYear export, which will aid in its reuse across the codebase. It's important to ensure that such changes are well-documented to maintain code quality and usability. This aligns with our previous decision to fix dead code in the calculate functions, emphasizing the importance of clarity and functionality in our calculations. Thank you for your contribution, and I look forward to your updates! |
📦 Next.js Bundle Size Report (Gzipped Sizes)✨ No significant bundle size changes detected. 📊 Summary of Totals
|
Aamod007
left a comment
There was a problem hiding this comment.
Adding export + JSDoc to daysInYear for reuse. Clean change. CI is failing — please fix.
Labels: level:beginner, quality:clean, type:docs, type:refactor, mentor:Aamod007
Aamod007
left a comment
There was a problem hiding this comment.
Re-reviewing after noting the only CI failure is Vercel authorization (not code-related). All other checks (Format · Lint · Typecheck · Test, Production Build, CodeQL) pass. Approving — the Vercel deploy auth is a repo-level config issue, not a code problem.
Summary of What Has Been Done
This PR addresses issue #8337.
See the corresponding issue for full details.
Closes #8337
Note: Please assign this PR to the
tmdeveloper007account.