Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add warn_by #14

Merged
merged 1 commit into from
Jun 27, 2022
Merged

Add warn_by #14

merged 1 commit into from
Jun 27, 2022

Conversation

leequarella
Copy link
Contributor

#12

  • Adds warn_by, allowing a user to add a date after which they will begin seeing warnings about the Todo.

searls
searls requested changes Jun 1, 2022
View reviewed changes
lib/todo_or_die.rb Outdated Show resolved Hide resolved
test/todo_or_die_test.rb Show resolved Hide resolved
@leequarella leequarella requested a review from searls June 27, 2022 14:41
@searls
Copy link
Owner

searls commented Jun 27, 2022

Woot! Looks great!

@searls searls merged commit 1ffca03 into searls:master Jun 27, 2022
@mikker
Copy link

mikker commented Jun 28, 2022

Adding some Lawful Neutral to a Chaotic Good lib.

@leequarella leequarella deleted the add_warn_by branch June 28, 2022 17:50
@bobmaerten bobmaerten mentioned this pull request Jul 1, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@searls searls Awaiting requested review from searls

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@leequarella @searls @mikker