Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 28, 2025

Rebased PR #267 onto current main. The workflow refactoring that replaced create_npm_release.yml already addressed the multi-branch release restriction. This PR preserves the CLI defaults that were part of the original change.

Changes

  • Added CLI option defaults: pathsAction'./action.yml', pathsReadme'README.md'
  • Updated type definition: Added default?: string to ArgvOptionProperties
  • Fixed tests: Removed path validation assertions (now optional with defaults) and isolated env vars in stringify test
argvOptions[ConfigKeys.pathsAction] = {
  alias: ['pathsAction', 'action'],
  type: 'string',
  describe: 'Path to the action.yml',
  default: './action.yml',  // Added
};

Users can now run the CLI without explicitly specifying action or readme paths when using standard locations.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@coderabbitai
Copy link

coderabbitai bot commented Dec 28, 2025

Important

Review skipped

Ignore keyword(s) in the title.

⛔ Ignored keywords (2)
  • WIP
  • DO NOT MERGE

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI and others added 2 commits December 28, 2025 18:59
Co-authored-by: Jamie-BitFlight <25075504+Jamie-BitFlight@users.noreply.github.com>
Co-authored-by: Jamie-BitFlight <25075504+Jamie-BitFlight@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix defaults to add default path for readme and action for cli usage fix(cli): add default values for action and readme path options Dec 28, 2025
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