Page MenuHomePhabricator

Fix the last remaining (?) continue inside switch
ClosedPublic

Authored by epriestley on Dec 22 2018, 2:12 PM.

Details

Summary

See https://discourse.phabricator-community.org/t/error-on-project-creation-or-edition-with-php7-3/2236/. This is the only remaining case that the linter rule in D19931 detected in libphutil, arcanist, or Phabricator.

Test Plan

Ran arc lint --everything ... in all three repositories, only hit this one.

Diff Detail

Repository
rP Phabricator
Branch
continue1
Lint
Lint OK
Unit
Unit Tests OK
Build Status
Buildable 21423
Build 29176: Run Core Tests
Build 29175: arc lint + arc unit

Event Timeline

epriestley created this revision.Dec 22 2018, 2:12 PM
epriestley requested review of this revision.Dec 22 2018, 2:14 PM
amckinley accepted this revision.Dec 24 2018, 9:20 PM
This revision is now accepted and ready to land.Dec 24 2018, 9:20 PM
This revision was automatically updated to reflect the committed changes.