Skip to content

🪲 [Fix]: Remove old inputs to workflows#91

Merged
Marius Storhaug (MariusStorhaug) merged 4 commits intomainfrom
fixWorkflow
Apr 21, 2025
Merged

🪲 [Fix]: Remove old inputs to workflows#91
Marius Storhaug (MariusStorhaug) merged 4 commits intomainfrom
fixWorkflow

Conversation

@MariusStorhaug
Copy link
Copy Markdown
Member

@MariusStorhaug Marius Storhaug (MariusStorhaug) commented Apr 21, 2025

Description

This pull request makes a small change to the .github/workflows/Process-PSModule.yml file by removing the with block, which previously specified SkipTests: SourceCode. This simplifies the workflow configuration.

  • Remove inputs from the .github/workflows/Process-PSModule.yml workflow.
  • Add skips for PSModule test for functions that are not covered by tests.

Type of change

  • 📖 [Docs]
  • 🪲 [Fix]
  • 🩹 [Patch]
  • ⚠️ [Security fix]
  • 🚀 [Feature]
  • 🌟 [Breaking change]

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

Copilot AI review requested due to automatic review settings April 21, 2025 08:19
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 pull request simplifies the workflow configuration by removing the outdated "SkipTests: SourceCode" input from the Process-PSModule workflow.

  • Removed the unused "with" block from the workflow file.
  • Simplified the workflow configuration for consistency with current practices.

@MariusStorhaug Marius Storhaug (MariusStorhaug) merged commit a105011 into main Apr 21, 2025
55 checks passed
@github-actions
Copy link
Copy Markdown

Module Utilities - 0.10.3 published to the PowerShell Gallery.

@github-actions
Copy link
Copy Markdown

GitHub release for Utilities v0.10.3 has been created.

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