-
-
Notifications
You must be signed in to change notification settings - Fork 60
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
Fixed Broken Links in Readme #431
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.
It appears that the LGTM has been subsumed into Github code scanning, and the LGTM.com website is no longer a thing, so yes, this link and image should be removed from the README.md file. It might be that it could be replaced with something else. I'm not sure. From what I'm reading, it appears that the functionality is now built into Github, but I'm not sure if that is something that has to be enabled for a particular repository.
The next link and image related to requires.io also appear to be defunct.
I'm not sure why there is an empty hyperlink around the blue python badge that precedes these two links, or what the meaning of that badge is.
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.
@Akhsuna07 I'm inclined to accept this, but you would have to accept the contributor license agreement first.
I haven't heard back from the requestor and they have not signed the CLA, so closing. |
Hey @aaronchantrill, I would love to work on this issue. |
@niranjan-kurhade, thank you for offering to help. I don't think either of them would have to be replaced with anything. Just removing them would be enough. Thank you, |
Description
Fixed the Broken Link in the readme file
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: