-
Notifications
You must be signed in to change notification settings - Fork 77
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
Downloading Fenix build with invalid platform raises an index error #664
Comments
PR #672 fixed only one case but we should still check early when checking command line arguments if the given platform is supported by the application. |
Hello @whimboo, this is the first time contributing to this code base. I would like to try my hand at it! |
Be welcome @randomicon00! It's great to see your interest. Let me know if you need further help to get started. You can ask here or as well join our Matrix channel. Thanks! |
@randomicon00 have you had the time to check the exiting code? Do you have any issues where you would need help with? Thanks |
Hi @whimboo! I've missed your first message (didn't get an email notification for it). I will start working on the task. I've already had a look at the source code before interacting here and I don't have any particular question. Thanks! |
@randomicon00 I wanted to check in with you to see if any other issues have come up while you’ve been working on it. If you just need a bit more time, that's perfectly fine — just let me know. I want to ensure we stay in touch. Thanks! |
Using the following arguments from a non Android platform causes the code in DailyScraper to fail with an
IndexError
because there is no such build for such a platform.The text was updated successfully, but these errors were encountered: