Skip to content

Add deadline and time expectation#16

Open
kimblim wants to merge 3 commits intomainfrom
kim/insert-deadline-time-estimate
Open

Add deadline and time expectation#16
kimblim wants to merge 3 commits intomainfrom
kim/insert-deadline-time-estimate

Conversation

@kimblim
Copy link
Copy Markdown
Collaborator

@kimblim kimblim commented Mar 23, 2026

No description provided.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c6a3f4a3-f0df-4e8e-8e7f-38d1dca30bdc

📥 Commits

Reviewing files that changed from the base of the PR and between b76a45e and e19671f.

⛔ Files ignored due to path filters (1)
  • .DS_Store is excluded by !**/.DS_Store
📒 Files selected for processing (1)
  • README.md
📜 Recent review details
🔇 Additional comments (1)
README.md (1)

42-45: LGTM! Clear communication of time expectations and deadlines.

The new section effectively communicates the assignment time expectations and submission deadline to candidates. The content is clear, professional, and well-formatted with proper markdown structure.


📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Added "Time & Deadline" section to README with assignment effort expectations (4–6 hours) and solution submission timeline requirements.

Walkthrough

A new "⏲️ Time & Deadline" section was added to the README.md file. This section includes information about the expected time investment for completing the assignment, specified as 4 to 6 hours, and establishes a requirement that the solution must be shared at least 48 hours before the scheduled interview. The change consists of 4 new lines with no deletions, and does not modify any code or public API declarations.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch kim/insert-deadline-time-estimate

Comment @coderabbitai help to get the list of available commands and usage tips.

msvarrer
msvarrer previously approved these changes Mar 23, 2026
@kimblim kimblim dismissed msvarrer’s stale review March 23, 2026 15:57

The merge-base changed after approval.

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