-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[DOC] Removes LGTM Badges from README.md #1651
Conversation
@mikofski this PR removes code quality row (containing LGTM badges) as requested by you |
Just wanted to note that this is related to GitHub retiring LGTM (see #1550 for further details). |
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.
Thanks @ooprathamm
Could you also remove the line "exclude .lgtm.yml" in the file "MANIFEST.in" and the file .lgtm.yml.
@ooprathamm a few more tasks to close this out:
|
@mikofski made the changes asked by you. |
docs/sphinx/source/reference
for API changes.docs/sphinx/source/whatsnew
for all changes. Includes link to the GitHub Issue with:issue:`num`
or this Pull Request with:pull:`num`
. Includes contributor name and/or GitHub username (link with:ghuser:`user`
).remote-data
) and Milestone are assigned to the Pull Request and linked Issue.