Page MenuHomePhorge
Differential D25078

Trigger: Capitalize most Buzzwords in our Weird Way as usual
Needs ReviewPublic
Actions

Authored by valerio.bozzolan on Mar 22 2023, 00:08.
Tags
None
Referenced Files
F418479: D25078.id.diff
Fri, Dec 8, 12:59
F417911: D25078.id352.diff
Fri, Dec 8, 07:00
F417377: D25078.id353.diff
Thu, Dec 7, 21:56
Unknown Object (File)
Thu, Dec 7, 19:14
Unknown Object (File)
Thu, Dec 7, 12:38
Unknown Object (File)
Thu, Nov 30, 21:51
Unknown Object (File)
Fri, Nov 24, 22:17
Unknown Object (File)
Fri, Nov 24, 22:17
View All 77 Files
Subscribers
Cigaryno
Matthew
speck
Sten
tobiaswiese
Tokens
"Hungry Hippo" token, awarded by valerio.bozzolan.

Details

Reviewers
Sten
Group Reviewers
O1: Blessed Committers
Test Plan
  • Open the Trigger Page from a Workboard Column
  • See that Most Things are Finally in this usual Weird Way

Diff Detail

Repository
rP Phorge
Branch
arcpatch-D25078_1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 737
Build 737: arc lint + arc unit

Event Timeline

valerio.bozzolan created this revision.Mar 22 2023, 00:08
Owners added a reviewer: O1: Blessed Committers.Mar 22 2023, 00:08
Herald added subscribers: Cigaryno, Matthew, tobiaswiese, speck. · View Herald TranscriptMar 22 2023, 00:08
valerio.bozzolan requested review of this revision.Mar 22 2023, 00:08
Harbormaster completed remote builds in B140: Diff 352.Mar 22 2023, 00:08
valerio.bozzolan updated this revision to Diff 353.Mar 22 2023, 00:10

update celerity to make the lint happy

Harbormaster completed remote builds in B141: Diff 353.Mar 22 2023, 00:10
valerio.bozzolan mentioned this in Z1: Phorge.Mar 22 2023, 14:34
valerio.bozzolan awarded a token.Mar 22 2023, 14:43
valerio.bozzolan updated this revision to Diff 361.Mar 22 2023, 18:07
  • also fix small typo "elmement"
Harbormaster completed remote builds in B149: Diff 361.Mar 22 2023, 18:07
valerio.bozzolan mentioned this in 2023-03-27.Mar 27 2023, 13:59
valerio.bozzolan mentioned this in Wikimedia Hackaton 2023 Athens Code Sprint.May 18 2023, 10:16
Sten accepted this revision.Aug 8 2023, 10:59
Sten subscribed.

Apart from the pre-existing 'Statues' typo, looks good to me...

src/applications/project/trigger/PhabricatorProjectTriggerManiphestStatusRule.php
27

Plural of 'Status' is 'Statuses'. 'Statues' is the plural of 'Statue', which is something different.

valerio.bozzolan updated this revision to Diff 1219.Aug 10 2023, 07:47

let's also fix a typo (thanks Sten)

Harbormaster completed remote builds in B737: Diff 1219.Aug 10 2023, 07:47

Revision Contents
Changeset List

Diff 1219

View Options

src/applications/project/trigger/PhabricatorProjectTriggerAddProjectsRule.php

Loading...
View Options

src/applications/project/trigger/PhabricatorProjectTriggerInvalidRule.php

Loading...
View Options

src/applications/project/trigger/PhabricatorProjectTriggerManiphestOwnerRule.php

Loading...
View Options

src/applications/project/trigger/PhabricatorProjectTriggerManiphestPriorityRule.php

Loading...
View Options

src/applications/project/trigger/PhabricatorProjectTriggerManiphestStatusRule.php

Loading...
View Options

src/applications/project/trigger/PhabricatorProjectTriggerPlaySoundRule.php

Loading...
View Options

src/applications/project/trigger/PhabricatorProjectTriggerRemoveProjectsRule.php

Loading...
View Options

src/applications/project/trigger/PhabricatorProjectTriggerUnknownRule.php

Loading...