Closed (fixed)
Project:
Automatic Updates
Version:
3.0.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
31 Aug 2023 at 16:45 UTC
Updated:
20 Sep 2023 at 18:24 UTC
Jump to comment: Most recent
Right now, we have explicit logic in StageBase to exclude the failure marker file from stage operations (in particular, the apply phase). I would argue that this is not awesome for two reasons:
Make FailureMarker an event subscriber that listens to CollectPathsToExcludeEvent, and adds the failure marker file as an excluded path. This will also simplify StageBase::apply() a tad.
Do it and add explicit test coverage.
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #3
phenaproximaComment #4
tedbowLooks good
Comment #5
phenaproximaComment #7
phenaproxima