Skip to content

Add '--fail-warnings' and '--skip-lint' flags to bundle publish#250

Merged
chrisghill merged 2 commits intomainfrom
chris/feature/add-linting-flags-to-publish
Sep 18, 2025
Merged

Add '--fail-warnings' and '--skip-lint' flags to bundle publish#250
chrisghill merged 2 commits intomainfrom
chris/feature/add-linting-flags-to-publish

Conversation

@chrisghill
Copy link
Copy Markdown
Member

This adds the new linting flags to the bundle_publish action.

Copy link
Copy Markdown

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 adds linting control flags to the bundle publish action, allowing users to configure how linting behaves during the bundle publishing process. It introduces two new command-line flags: --fail-warnings to treat linting warnings as failures, and --skip-lint to bypass linting entirely.

  • Added two new input parameters (fail-warnings and skip-lint) to the bundle_publish action
  • Updated the TypeScript implementation to conditionally add these flags to the CLI command
  • Updated example workflow to use the latest action version (v5)

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
bundle_publish/action.yml Added new input parameters for linting control and updated descriptions
src/bundle_publish.ts Implemented logic to conditionally add linting flags to the CLI command
example_workflows/md_bundle_publish.yaml Updated action versions from v4 to v5

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

@chrisghill chrisghill merged commit 40c0501 into main Sep 18, 2025
6 checks passed
@chrisghill chrisghill deleted the chris/feature/add-linting-flags-to-publish branch September 18, 2025 21:24
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.

3 participants