Skip to content

Conversation

@AgbajeOlayiwola
Copy link
Contributor

Added the edit tasks function run with cargo run -- edit [index] 'new_text'

@MrGranday
Copy link
Owner

You should trim new_text to avoid empty edits ("" or spaces).
Consider returning a Result<(), String> instead of printing errors directly. That way tests can assert behavior.
Please add unit tests for both success and invalid index cases.

@AgbajeOlayiwola
Copy link
Contributor Author

i have done both and the pr has been updaed

@AgbajeOlayiwola AgbajeOlayiwola mentioned this pull request Sep 2, 2025
@AgbajeOlayiwola
Copy link
Contributor Author

hello can you check this PR all fixes have been implemented including tests for edit and add deadline

@MrGranday MrGranday merged commit 301613e into MrGranday:main Sep 4, 2025
1 check passed
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