Skip to content

[CleanRepo] Update action.yml file with new option - #809

Merged
gewarren merged 6 commits into
dotnet:mainfrom
gewarren:parse-string
Nov 4, 2026
Merged

gewarren merged 6 commits into
dotnet:mainfrom
gewarren:parse-string

Conversation

@gewarren

@gewarren gewarren commented Nov 4, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI balanced review requested due to automatic review settings November 4, 2026 21:45

Copilot AI 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.

🟡 Changes recommended

Exclusion handling can ignore command-line overrides and silently misparse malformed arrays, allowing files intended to be excluded to be deleted by default.

2 open issues
What changed in this PR

Adds an exclusion-list input to the CleanRepo GitHub Action so callers can protect selected include files from orphan cleanup.

Changes:

  • Adds exclude_include_files and forwards it to CleanRepo.
  • Adds parsing for JSON arrays and comma-, semicolon-, or newline-separated exclusion lists.
  • Updates the action’s function description and simplifies media-list initialization.
File Description
cleanrepo/​Program.cs Parses string-based exclusions and simplifies media-list initialization.
cleanrepo/​action.yml Exposes and forwards the exclusion input; updates the function description.

Review effort: Balanced

Comment thread cleanrepo/Program.cs Resolved
Comment thread cleanrepo/Program.cs Resolved
gewarren and others added 4 commits November 4, 2026 13:49
Enhance error handling for JSON deserialization.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI 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.

🟢 Approval recommended

There are no active review issues for the include-file exclusion input change.

No open issues 🎉

2 resolved since last review

Review effort: Balanced

@gewarren
gewarren merged commit 1792e49 into dotnet:main Nov 4, 2026
7 checks passed
@gewarren
gewarren deleted the parse-string branch November 4, 2026 22:50
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