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
Extra check for incompatible options #63
Comments
@XhmikosR totally agree and have been on my mind as well. Would you like to submit a PR for this? |
Sorry, I don't have a lot of free time to tackle this. I just wanted to
make an issue to track this. :)
…On Thu, Feb 6, 2020, 21:53 Liran Tal ***@***.***> wrote:
@XhmikosR <https://github.com/XhmikosR> totally agree and have been on my
mind as well.
It should also be straightforward to do with yargs configuration options.
You can detect if both are being used and then fail, or better yet, in my
opinion schemes should override https.
Would you like to submit a PR for this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#63?email_source=notifications&email_token=AACVLNN32KYSTRTMNUK2VNDRBRTCLA5CNFSM4KQZP77KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELARUQQ#issuecomment-583080514>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACVLNIXQWAMJKYM5LLLXE3RBRTCLANCNFSM4KQZP77A>
.
|
No worries at all, I'll shoot in a PR for that soon enough and ping you then ❤️ |
@XhmikosR how does this look like:
|
@lirantal yup, that works! |
Great. Merging away 🎉 |
Is your feature request related to a problem? Please describe.
#23 (comment)
Describe the solution you'd like
Since the above works without the
validate-https
flag, maybe there should be some checks in place for potentially incompatible options.The text was updated successfully, but these errors were encountered: