Skip to content

Develop #219

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 28, 2025
Merged

Develop #219

merged 3 commits into from
Jun 28, 2025

Conversation

ptr727
Copy link
Owner

@ptr727 ptr727 commented Jun 28, 2025

No description provided.

@Copilot Copilot AI review requested due to automatic review settings June 28, 2025 01:13
Copy link

@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 PR standardizes and enhances the CI/CD workflows by renaming and rescheduling the release-version updater, adding a dedicated PR test workflow, and refining the build-publish pipeline along with editorconfig updates.

  • Rename and adjust schedule for the release versions workflow
  • Introduce TestPullRequest.yml for unit tests and Docker build matrix
  • Remove redundant .NET SDK setup, add parallelization, and add LF rule for s6-overlay in .editorconfig

Reviewed Changes

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

File Description
.github/workflows/UpdateReleaseVersions.yml Renamed workflow title, job names, commit messages, and updated cron schedule
.github/workflows/TestPullRequest.yml New workflow for pull-request unit tests, matrix setup, and Docker image builds
.github/workflows/BuildPublishPipeline.yml Removed .NET SDK setup step; added max-parallel: 4 to matrix strategy
.editorconfig Added LF line-ending override for Docker/s6-overlay/**
Comments suppressed due to low confidence (3)

.github/workflows/BuildPublishPipeline.yml:34

  • The .NET SDK setup step was removed but later steps still rely on dotnet commands. Re-add actions/setup-dotnet@v4 with dotnet-version: 9.x before any dotnet CLI invocation to prevent failures.
      # https://github.com/marketplace/actions/checkout

.github/workflows/UpdateReleaseVersions.yml:1

  • [nitpick] The workflow title changed to 'Update Released Versions' but the branch (update-version) and some messages still reference 'release versions'. Consider standardizing on either 'release' or 'released' across names and messages to avoid confusion.
name: Update Released Versions

.editorconfig:25

  • [nitpick] The new section targets Docker/s6-overlay/** for LF endings—consider adding a brief comment explaining why this override is necessary to aid future maintainers.
[Docker/s6-overlay/**]

@ptr727 ptr727 merged commit a41368d into main Jun 28, 2025
16 checks passed
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.

1 participant