-
Notifications
You must be signed in to change notification settings - Fork 1
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
Update dependencies #1092
Update dependencies #1092
Conversation
Oof not sure what's up with that build failure |
@kylelaker-ccpo okay, so it's not just me I was trying to wrap my mind around the issue, no luck either. |
It looks like maybe |
Ahh yeah yargs/yargs-parser#452 seems related |
yeah, that definitely is the same error we're getting. Do we just wait for that issue to be fixed? Or is there something else we can do? |
This includes additional dependency updates, including one that hopefully fixes `yargs-parser` related errors in `tsc`.
How did I end up with merge conflicts when I did that? Okay... uhh let me work that out. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Thanks for the additions! Looks like the issue was fixed for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm good with just approving. It's dependencies only and if we're both good with the changes, I think that's sufficient.
No description provided.