Skip to content

Add program to determine days in a month - #2

Merged
NachoOFC merged 1 commit into
mainfrom
NachoOFC-patch-2
Sep 14, 2026
Merged

NachoOFC merged 1 commit into
mainfrom
NachoOFC-patch-2

Conversation

@NachoOFC

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings September 14, 2026 14:04
@NachoOFC
NachoOFC merged commit b22193a into main Sep 14, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

February leap-year handling or an explicit non-leap-year contract is still needed.

Pull request overview

Adds a standalone Java console program that determines the number of days in a selected month.

Changes:

  • Reads a month number from standard input.
  • Handles 30-day, 31-day, February, and invalid month cases.
File summaries
File Summary
dias_mes.java Implements month validation and day-count selection. February is hard-coded to 28, which is incorrect for leap years; accept a year or state the non-leap-year assumption explicitly.
Review details

Suppressed comments (1)

dias_mes.java:18

  • February is hard-coded to 28, so this program reports the wrong number of days for February in leap years. Since the input only identifies the month, please either accept a year and apply the leap-year rule, or make the non-leap-year assumption explicit in the program's contract/output.
                dias = 28; // Ignorando años bisiestos para simplicidad
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

2 participants