Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion .editorconfig
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ indent_size = 2
end_of_line = crlf
indent_size = 2

# Json files
# Json and JsonC files
[*.{json,jsonc}]
end_of_line = crlf

Expand All @@ -48,9 +48,15 @@ end_of_line = lf
[*.{cmd,bat,ps1}]
end_of_line = crlf

# --- .NET-only below: C# and ReSharper style. Everything above is the line-ending
# governance every derived repo carries; a non-.NET repo may drop from here down. ---

# C# files
[*.cs]
end_of_line = crlf
# Suppressions follow CODESTYLE.md "Analyzer Diagnostics and Suppressions": prefer a
# [SuppressMessage] attribute or the owning project's .editorconfig; relax a rule
# repo-wide here only when it applies to every project (never a brownfield batch).
dotnet_diagnostic.IDE0055.severity = none
dotnet_analyzer_diagnostic.severity = suggestion

Expand Down
525 changes: 42 additions & 483 deletions .github/copilot-instructions.md

Large diffs are not rendered by default.

17 changes: 6 additions & 11 deletions .github/workflows/build-executable-task.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,6 @@ on:
required: false
type: boolean
default: false
outputs:
# Output of the uploaded artifact id
artifact-id:
value: ${{ jobs.upload-build-artifacts.outputs.artifact-id }}

jobs:

Expand Down Expand Up @@ -73,16 +69,15 @@ jobs:
with:
name: publish-${{ inputs.branch }}-${{ matrix.runtime }}
path: ${{ runner.temp }}/publish
retention-days: 1

# Smoke builds only need the per-runtime compile to succeed (fast PR
# feedback) — the zipped, downloadable artifact is a release concern, so
# skip the aggregation entirely on smoke. The `artifact-id` output is then
# empty, which is fine because the GitHub release job never runs on smoke.
# feedback) - the zipped, downloadable artifact is a release concern, so
# skip the aggregation entirely on smoke. The release job collects the
# release-asset-<branch>-* artifacts by pattern, so no artifact-id is needed.
upload-build-artifacts:
name: Upload matrix build artifacts job
if: ${{ !inputs.smoke }}
outputs:
artifact-id: ${{ steps.artifact-upload-step.outputs.artifact-id }}
runs-on: ubuntu-latest
needs: [ build-executable-matrix ]

Expand All @@ -99,8 +94,8 @@ jobs:
run: 7z a -t7z ${{ runner.temp }}/PlexCleaner.7z ${{ runner.temp }}/publish/*

- name: Upload build artifacts step
id: artifact-upload-step
uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
with:
name: executable-build-${{ inputs.branch }}
name: release-asset-${{ inputs.branch }}-executable
path: ${{ runner.temp }}/PlexCleaner.7z
retention-days: 1
7 changes: 5 additions & 2 deletions .github/workflows/build-release-task.yml
Original file line number Diff line number Diff line change
Expand Up @@ -103,10 +103,13 @@ jobs:
with:
ref: ${{ needs.get-version.outputs.GitCommitId }}

- name: Download executable build artifacts step
# Collect every release-asset-<branch>-* artifact by pattern, so the
# release job never names a build job and stays reusable as targets change.
- name: Download release asset artifacts step
uses: actions/download-artifact@3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c # v8.0.1
with:
artifact-ids: ${{ needs.build-executable.outputs.artifact-id }}
pattern: release-asset-${{ inputs.branch }}-*
merge-multiple: true
path: ./Publish

# The weekly publisher re-runs even when a branch has no new commits, so
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/build-toolversions-task.yml
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ jobs:
with:
# Branch-suffixed so both matrix legs coexist; the inner filename
# stays `latest.ver` for main (the m4 `include({{latest.ver}})`
# token), consumed by build-dockerreadme-task.yml.
# token), consumed by publish-docker-readme-task.yml.
name: versions-${{ inputs.branch }}
path: ${{ runner.temp }}/versions/${{ inputs.branch == 'main' && 'latest.ver' || 'develop.ver' }}
retention-days: 1
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Create Docker README.md task
name: Publish Docker Hub readme task

on:
workflow_call:
Expand All @@ -15,7 +15,7 @@ on:
jobs:

docker-readme:
name: Create Docker README.md job
name: Publish Docker Hub readme job
# Render only for main: there is a single Docker Hub README, and the m4
# template includes the `latest` image's tool versions.
if: ${{ inputs.branch == 'main' }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/publish-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -95,13 +95,13 @@ jobs:
branch: ${{ matrix.branch }}

docker-readme:
name: Create Docker README.md job
name: Publish Docker Hub readme job
needs: [setup, tool-versions]
if: ${{ needs.setup.outputs.publish == 'true' }}
strategy:
matrix:
branch: ${{ fromJSON(needs.setup.outputs.branches) }}
uses: ./.github/workflows/build-dockerreadme-task.yml
uses: ./.github/workflows/publish-docker-readme-task.yml
secrets: inherit
with:
# The task self-gates to `main`; the develop leg is a no-op.
Expand Down
3 changes: 2 additions & 1 deletion .markdownlint-cli2.jsonc
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@
"MD033": false,
// Require fenced code blocks over the legacy 4-space-indented style.
"MD046": { "style": "fenced" },
// Wide tables are intentional where wrapping cells breaks GitHub rendering.
// MD060 (table column style) is not enforced - allow both compact
// (`|a|b|`) and padded (`| a | b |`) table pipe spacing.
"MD060": false
},
"gitignore": true
Expand Down
1 change: 1 addition & 0 deletions .vscode/tasks.json
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@
"showReuseMessage": false,
"clear": false
},
"dependsOrder": "sequence",
"dependsOn": [
"CSharpier Format",
".Net Build"
Expand Down
Loading
Loading