Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: warn when there is a ytdl-core update (#779)
* Alert if ytdl-core is old * Add variables and pacify the linter ...hopefully. why is readability so important in a line that serves no more than utility? * Cause the linter to not be angry, again I'd really like to see all the checks pass for once. This code is absolutely readable, I dare you to say otherwise. * Change console.log to console.error for the linter * Get version from package.json As suggested by fent Co-authored-by: fent <fentbox@gmail.com> * Allow console.warn * Move the warn code to getInfo() * Add env variable support env var UPDATE if not set: check twice daily settings: 'always', 'never' * Make requested changes Specifically: change env var to YTDL_NO_UPDATE which takes true or false Drop code outside of functions (that's what I think you meant by that request. will it still run during long-uptime?) Clearer message, calc ms number. gonna drop the env definition into nock, hold on * Add env definition as requested * Fix the eslintrc * Indent properly I'm migrating to a new computer so I'm doing this all from GH web, it's a nightmare * Appease the linter once more I fixed this already, huh whuh? Useful commit comment goes here * Maybe this will be correct indentation? If not, please fix it for me. * Made changes you requested, and some others: Fixed the indents for eslint (hopefully, it seems to dislike me) Implemented request of not checking until the first 12 hrs are over Made it a function, and called it inside the getInfo cache thing directly above it. Also exported it so it could be used programatically (seems like a good idea?) Please lmk if there's anything else you'd like to see. * Fixing silly mistakes subtitle: my brain is big and smart coming to theaters near y'all * so that's why it wanted a user-agent * Make line shorter for the linter Co-authored-by: fent <fentbox@gmail.com>
- Loading branch information