Skip to content

Conversation

MariusStorhaug
Copy link
Member

@MariusStorhaug MariusStorhaug commented Oct 4, 2025

This pull request updates and simplifies the repository's GitHub workflow and configuration files, focusing on workflow consolidation, dependency management, and configuration cleanup. The main changes include merging the nightly and pull request workflows, updating workflow versions, improving Dependabot labeling, and removing unused configuration files.

Workflow consolidation and updates:

  • Combined the nightly scheduled workflow and the pull request workflow into a single .github/workflows/Process-PSModule.yml file, which now runs on both a nightly schedule and on pull requests to main. The workflow was also updated to use workflow.yml@v5 and to inherit secrets, simplifying maintenance. [1] [2] [3]
  • Removed the separate .github/workflows/Nightly-Run.yml and .github/workflows/Linter.yml files, consolidating workflow logic and eliminating redundant jobs. [1] [2]

Dependency management:

  • Updated .github/dependabot.yml to automatically add dependencies and github-actions labels to PRs created for GitHub Actions updates, improving visibility and triage of dependency updates.

Configuration cleanup:

  • Removed the .github/linters/.jscpd.json file, indicating that JSCPD duplicate code detection is no longer configured or required in this repository.
  • Added a new .github/PSModule.yml configuration file to set a code coverage target for tests, supporting the Process-PSModule workflow.

@MariusStorhaug MariusStorhaug self-assigned this Oct 4, 2025
@Copilot Copilot AI review requested due to automatic review settings October 4, 2025 07:48
@MariusStorhaug MariusStorhaug requested a review from a team as a code owner October 4, 2025 07:48
Copy link
Contributor

@Copilot 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 updates the repository's CI/CD infrastructure by bumping the Process-PSModule workflow to version 5 and consolidating GitHub Actions workflows. The changes streamline the automation pipeline by removing redundant workflow files while adding proper configuration for the updated workflow.

  • Updated Process-PSModule workflow to v5 with simplified secret inheritance
  • Consolidated nightly runs and linting into the main workflow by removing separate files
  • Added configuration file for the Process-PSModule workflow with code coverage settings

Reviewed Changes

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

Show a summary per file
File Description
.github/workflows/Process-PSModule.yml Updated to v5, added schedule triggers, and simplified secrets management
.github/workflows/Nightly-Run.yml Removed standalone nightly workflow (consolidated into main workflow)
.github/workflows/Linter.yml Removed standalone linter workflow (consolidated into main workflow)
.github/linters/.jscpd.json Removed unused duplicate code detection configuration
.github/dependabot.yml Added labels for better categorization of dependency updates
.github/PSModule.yml Added configuration file for Process-PSModule workflow with code coverage target

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

@MariusStorhaug MariusStorhaug merged commit 34c013c into main Oct 4, 2025
143 of 148 checks passed
@MariusStorhaug MariusStorhaug deleted the bump branch October 4, 2025 12:41
@github-project-automation github-project-automation bot moved this from Todo to Done in GitHub PowerShell Module Oct 4, 2025
Copy link
Contributor

github-actions bot commented Oct 4, 2025

Module GitHub - 0.40.3 published to the PowerShell Gallery.

Copy link
Contributor

github-actions bot commented Oct 4, 2025

GitHub release for GitHub v0.40.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
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant