Page MenuHomePhabricator
Differential D21689

Modularize HarbormasterBuildTransaction
ClosedPublic
Actions

Authored by epriestley on Jul 15 2021, 9:05 PM.
Tags
None
Referenced Files
F15418745: D21689.id51702.diff
Fri, Mar 21, 1:12 AM
F15407600: D21689.diff
Tue, Mar 18, 6:37 PM
F15392521: D21689.diff
Sat, Mar 15, 3:41 PM
F15375409: D21689.diff
Wed, Mar 12, 11:05 PM
F15374555: D21689.id.diff
Wed, Mar 12, 7:55 PM
F15367926: D21689.diff
Tue, Mar 11, 10:13 PM
F15321394: D21689.diff
Thu, Mar 6, 10:56 PM
Unknown Object (File)
Mon, Feb 24, 11:03 AM
View All Files
Subscribers
Restricted Owners Package

Details

Reviewers
None
Maniphest Tasks
T13072: Merge Harbormaster BuildCommand into BuildMessage
Commits
rP6dfea0adaddc: Modularize HarbormasterBuildTransaction
Summary

Ref T13072. Update "HarbormasterBuild" to use modern modular transactions.

Test Plan
  • Aborted, restarted, paused, and resumed a build.
  • Used bin/harbormaster restart.
  • Grepped for use of old "::TYPE_COMMAND" constant, didn't find any hits.

Diff Detail

Repository
rP Phabricator
Branch
build6
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/harbormaster/xaction/build/HarbormasterBuildMessageTransaction.php:82XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 25454
Build 35176: Run Core Tests
Build 35175: arc lint + arc unit

Event Timeline

epriestley created this revision.Jul 15 2021, 9:05 PM
Owners added a subscriber: Restricted Owners Package.Jul 15 2021, 9:05 PM
Harbormaster completed remote builds in B25454: Diff 51651.Jul 15 2021, 9:07 PM
epriestley requested review of this revision.Jul 15 2021, 9:07 PM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 21 2021, 9:17 PM
Closed by commit rP6dfea0adaddc: Modularize HarbormasterBuildTransaction (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP6dfea0adaddc: Modularize HarbormasterBuildTransaction.

Revision Contents
Changeset List

Diff 51651

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/harbormaster/editor/HarbormasterBuildTransactionEditor.php

Loading...
View Options

src/applications/harbormaster/engine/HarbormasterBuildEngine.php

Loading...
View Options

src/applications/harbormaster/management/HarbormasterManagementRestartWorkflow.php

Loading...
View Options

src/applications/harbormaster/storage/HarbormasterBuildTransaction.php

Loading...
View Options

src/applications/harbormaster/xaction/build/HarbormasterBuildMessageTransaction.php

Loading...
View Options

src/applications/harbormaster/xaction/build/HarbormasterBuildTransactionType.php

Loading...
Phabricator · Built by Phacility