ippsec.github.io icon indicating copy to clipboard operation
ippsec.github.io copied to clipboard

Bug in using argparse and multiple repetition of default values

Open StellarClown opened this issue 3 years ago • 1 comments

Using the script I noticed that there was a bug in the commit automation script checking. The parameters had been reversed. Also, the commit flag, so to speak, the -g had wrong parameters that forced it to have an argument.

Furthermore, as explained in the issue, there is a need to change the readme, as it is incorrectly recommended to use the script. I recommend the following change:

python yt_crawl.py -a "API_KEY_HERE" -g

StellarClown avatar Oct 11 '22 10:10 StellarClown

Checking the sources, I think it is also useless to keep the yt_data.py file, as it appears to be an outdated version of this file. I also recommend updating the json file.

StellarClown avatar Oct 11 '22 15:10 StellarClown