Skip to content

🚀[Feature]: Add Notice_Mode input to control test completion notifications - #21

Merged
Marius Storhaug (MariusStorhaug) merged 10 commits into
mainfrom
debugPath
Oct 12, 2025
Merged

🚀[Feature]: Add Notice_Mode input to control test completion notifications#21
Marius Storhaug (MariusStorhaug) merged 10 commits into
mainfrom
debugPath

Conversation

@MariusStorhaug

@MariusStorhaugMarius Storhaug (MariusStorhaug) commented Oct 12, 2025

Copy link
Copy Markdown
Member

This release introduces a new Notice_Mode input parameter to control when GitHub notices are displayed for test completion. By default, notices are now only shown on test failure, providing more flexibility in managing action output verbosity.

What's New

  • New Input: Notice_Mode - Controls when GitHub notices are displayed for test completion:
    • Full: Show notices on both success and failure.
    • Failed (default): Show notices only on failure.
    • None: Disable notices entirely.
      This provides more flexibility in managing action output verbosity.

Changes

  • Updated action.yml to define the new input and pass it via environment variables.
  • Updated README.md with input documentation.

CopilotAI review requested due to automatic review settings October 12, 2025 20:43

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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 a new Notice_Mode input parameter to control when test completion notifications are displayed. The feature allows users to configure notification behavior with options for "Full" (show on success and failure), "Failed" (show only on failure), or "None" (disable notices).

  • Added Notice_Mode input parameter with description and default value of 'Failed'
  • Updated action configuration to pass the new parameter to the test execution step
  • Updated documentation table to include the new input parameter

Reviewed Changes

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

FileDescription
action.ymlAdded Notice_Mode input definition and passed it to the PSScriptAnalyzer test step
README.mdUpdated inputs documentation table to include the new Notice_Mode parameter
scripts/main.ps1Modified settings file resolution logic with debug output (unrelated to Notice_Mode feature)

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

Comment threadscripts/main.ps1
Comment threadscripts/main.ps1
@MariusStorhaugMarius Storhaug (MariusStorhaug) changed the title 🚀[Feature]: Add Notice_Mode input to control test completion notifications🚀[Feature]: Add Notice_Mode input to control analysis completion notificationsOct 12, 2025
@MariusStorhaugMarius Storhaug (MariusStorhaug) changed the title 🚀[Feature]: Add Notice_Mode input to control analysis completion notifications🚀[Feature]: Add Notice_Mode input to control test completion notificationsOct 12, 2025
@MariusStorhaugMarius Storhaug (MariusStorhaug) changed the title 🚀[Feature]: Add Notice_Mode input to control test completion notifications🚀[Feature]: Add Notice_Mode input to control test completion notificationsOct 12, 2025
CopilotAI review requested due to automatic review settings October 12, 2025 21:08

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.


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

Comment threadscripts/main.ps1
@MariusStorhaug
Marius Storhaug (MariusStorhaug) merged commit 70815e8 into mainOct 12, 2025
22 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@MariusStorhaug