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

Check and download from all sites #73

Merged
merged 17 commits into from
Jun 14, 2021

Conversation

Senth
Copy link
Owner

@Senth Senth commented Jun 14, 2021

Now checks and downloads from all sites

What changed?

  • Added configuring for multiple sites
  • configure command changed because of this
  • Now sets the site id again

Testing

  • Added unit tests
  • Manual install
  • Manual update

Related Issues

Fixes #27
Fixes #62 as it always downloads from all sites

@lgtm-com
Copy link

lgtm-com bot commented Jun 14, 2021

This pull request introduces 7 alerts when merging e256452 into 19b4991 - view on LGTM.com

new alerts:

  • 4 for Unused import
  • 1 for Statement has no effect
  • 1 for Empty except
  • 1 for Testing equality to None

@lgtm-com
Copy link

lgtm-com bot commented Jun 14, 2021

This pull request introduces 2 alerts when merging 8df77b5 into 19b4991 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@Senth Senth merged commit 984bc55 into main Jun 14, 2021
@Senth Senth deleted the 27-check-all-sites-and-download-the-latest branch June 14, 2021 10:17
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.

Let the user configure preferred repository Check all sites and download the latest
1 participant