Paths

Table of Contentst

Differential D25679

Remove unused variable from HeraldEngine
ClosedPublic
Actions

Authored by aklapper on Jun 4 2024, 09:34.

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Commits

rPa8af0c6e5119: Remove unused variable from HeraldEngine

Summary

$activeRule is not used anywhere.

Test Plan

Grep the code.

Diff Detail

Repository

rP Phorge

Branch

rmHeraldActiveRule (branched from master)

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 1317
Build 1317: arc lint + arc unit

Event Timeline

aklapper created this revision.Jun 4 2024, 09:34

Owners added a reviewer: O1: Blessed Committers.Jun 4 2024, 09:34

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptJun 4 2024, 09:34

aklapper requested review of this revision.Jun 4 2024, 09:34

Harbormaster completed remote builds in B1317: Diff 2067.Jun 4 2024, 09:34

Is anything extending HeraldEngine trying direct access to that?

grep -RF -- '->activeRule' .

No results.

This revision is now accepted and ready to land.Jun 4 2024, 10:24

Closed by commit rPa8af0c6e5119: Remove unused variable from HeraldEngine. · Explain WhyJun 5 2024, 08:30

This revision was automatically updated to reflect the committed changes.

aklapper added a commit: rPa8af0c6e5119: Remove unused variable from HeraldEngine.

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

herald/

engine/

HeraldEngine.php

1 line

O1: Blessed Committers

Diff 2067

View Options

src/applications/herald/engine/HeraldEngine.php

Remove unused variable from HeraldEngineClosedPublicActions