Skip to content

Update Evan at NRI to be past-tense with a range of time.#140

Open
perkee wants to merge 1 commit intojah2488:masterfrom
perkee:master
Open

Update Evan at NRI to be past-tense with a range of time.#140
perkee wants to merge 1 commit intojah2488:masterfrom
perkee:master

Conversation

@perkee
Copy link

@perkee perkee commented Jan 9, 2026

Please review our Contribution Guidelines to ensure your changes abides by it.

Review your changes and complete the checklist below.

  • Company name
  • Company description
  • Company location
  • GitHub link
  • Sorted alphabetically
  • Lines wrapped at 80 chars

Summary by CodeRabbit

  • Documentation
    • Updated factual information regarding a founder's employment history, correcting the timeline to reflect past employment from 2016 through 2020 rather than current employment status.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 9, 2026

📝 Walkthrough

Walkthrough

A factual correction to the README.md file updates the employment status of Elm's creator at NoRedInk, changing from present tense to specify the employment period from 2016 through 2020.

Changes

Cohort / File(s)
Documentation Update
README.md

Details: Updated employment tenure information—changed "is a NoRedInk employee" to "was a NoRedInk employee from 2016 through 2020"

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

📄 A fact was spun with gentle care,
The timeline now rings crystal clear,
From 2016 through twenty-twenty's end,
The truth's corrected, paragraph by friend! 🐰✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title accurately describes the main change: updating a factual statement about Evan from present to past tense with a specific time range.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@perkee perkee changed the title Update Evan at NRI Update Evan at NRI to be past-tense with a range of time. Jan 9, 2026
@perkee
Copy link
Author

perkee commented Jan 9, 2026

Not at all advocating for keeping the range. If it's better to do just "was an employee at NoRedInk" or just take out the Evan line altogether that makes sense to me.

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.

1 participant