Paths

Table of Contentst

Remove unused variable from HeraldEngine
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F3801629: D25679.1745958851.diff
	Mon, Apr 28, 20:34

	F3746364: D25679.1745768066.diff
	Sat, Apr 26, 15:34

	F3712092: D25679.1745671118.diff
	Fri, Apr 25, 12:38

	F3702099: D25679.1745617298.diff
	Thu, Apr 24, 21:41

	F3605059: D25679.1745251485.diff
	Sun, Apr 20, 16:04

	F3417915: D25679.1744616604.diff
	Sun, Apr 13, 07:43

	F3390882: D25679.1744472589.diff
	Fri, Apr 11, 15:43

	F3372507: D25679.1744250134.diff
	Wed, Apr 9, 01:55

Subscribers

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