-
Notifications
You must be signed in to change notification settings - Fork 28
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.
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
Creating delete_links for issue #79 provide delete_link function #91
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to be pretty good!
For some reason the tests didn't trigger automatically and I might have to check on that later, but can you run them and paste them here?
below the tests output
|
After updating Poetry... ALL tests PASSED 👍
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR for fixing Issue #79.
function
delete_link
created and tests passed successfully.