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.

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

Cleanup #105

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Cleanup #105

wants to merge 17 commits into from

Conversation

matteoferla
Copy link

Some fixes to the repo

Removed

  • Removed invalid docker code

Fixed

  • Moved doc files out of package
  • Cleaned up installation disorder
  • Changed version from 0.0.5.1 to 0.6 as pypi has 0.1.1

Added

  • MANIFEST.in for two files which are required by the package
  • Submodule quick for a quick download of the data as it appears on the site, which is what most folks want anyway

@lgtm-com
Copy link

lgtm-com bot commented Dec 5, 2022

This pull request introduces 1 alert when merging ed1d71e into b09a4b9 - view on LGTM.com

new alerts:

  • 1 for Unnecessary 'else' clause in loop

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

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.

1 participant