Skip to content

Conversation

@keneanung
Copy link
Contributor

This is a fix necessary to let the CI jobs run again. I missed this breaking change from the changelog when updating yargs to version 18.

This is a fix necessary to let the CI jobs run again. I missed this breaking change from the changelog when updating yargs to version 18.
@keneanung keneanung requested review from Copilot and vadi2 October 1, 2025 16:18
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes compatibility with yargs version 18 by adapting the library initialization syntax. The breaking change requires passing process.argv explicitly to the yargs constructor instead of relying on the previous implicit behavior.

  • Updates yargs initialization to explicitly pass process.argv parameter

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link

github-actions bot commented Oct 1, 2025

Warnings
⚠️ Main map file not edited. Are you sure you uploaded the file with the correct name?

Generated by 🚫 dangerJS against a876560

@keneanung keneanung merged commit a8c58b3 into development Oct 6, 2025
3 of 4 checks passed
@keneanung keneanung deleted the fix-yargs-usage branch October 6, 2025 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants