Skip to content

CI validation (draft): preserve plugins during failed updates - #167

Closed
Blackspirits wants to merge 4 commits into
ci-base-subtitleedit-20261111from
fix/plugin-install-safety-20261111
Closed

Blackspirits wants to merge 4 commits into
ci-base-subtitleedit-20261111from
fix/plugin-install-safety-20261111

Conversation

@Blackspirits

Copy link
Copy Markdown
Owner

CI validation only — fork-local draft PR, not intended for merge.

Temporary baseline matches current upstream SubtitleEdit/subtitleedit@c0e7ac3e0ef97bcc1b6efdc24a995f7ea94e5909.

Addresses two confirmed installation risks: staging folders showing as installed plugins and deletion of working plugins before successful replacement. Adds destination-path validation, manifest checks, rollback on publish failures, and focused tests.

Not yet verified: .NET build, focused tests, broader regression suite, independent review. This draft exists solely to exercise GitHub Actions; do not merge.

Upstream contribution will require a separate reviewed PR after validation.

Copy link
Copy Markdown
Owner Author

Superseded by fork-only CI Draft #171, reconstructed on the current upstream main commit 5e673e4 without overlapping upstream file changes. The new branch HEAD f4bbc30c passed all four Linux CI test jobs. This superseded PR is closed without merge to avoid duplicate review/CI branches; the new PR remains Draft, requires independent review and (for UI) Windows visual validation.

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