diff --git a/scripts/setup/manage_garbage.php b/scripts/setup/manage_garbage.php
index ba727eab60..326730375e 100755
--- a/scripts/setup/manage_garbage.php
+++ b/scripts/setup/manage_garbage.php
@@ -5,7 +5,7 @@ $root = dirname(dirname(dirname(__FILE__)));
 require_once $root.'/scripts/__init_script__.php';
 
 $args = new PhutilArgumentParser($argv);
-$args->setTagline(pht('manage garbage colletors'));
+$args->setTagline(pht('manage garbage collectors'));
 $args->setSynopsis(<<<EOSYNOPSIS
 **garbage** __command__ [__options__]
     Manage garbage collectors.
diff --git a/src/aphront/httpparametertype/AphrontHTTPParameterType.php b/src/aphront/httpparametertype/AphrontHTTPParameterType.php
index 995556a6b9..78a62a663c 100644
--- a/src/aphront/httpparametertype/AphrontHTTPParameterType.php
+++ b/src/aphront/httpparametertype/AphrontHTTPParameterType.php
@@ -160,7 +160,7 @@ abstract class AphrontHTTPParameterType extends Phobject {
   /**
    * Call another type's existence check.
    *
-   * This method allows a type to reuse the exitence behavior of a different
+   * This method allows a type to reuse the existence behavior of a different
    * type. For example, a "list of users" type may have the same basic
    * existence check that a simpler "list of strings" type has, and can just
    * call the simpler type to reuse its behavior.
diff --git a/src/applications/almanac/editor/AlmanacDeviceEditor.php b/src/applications/almanac/editor/AlmanacDeviceEditor.php
index 64b8de7bad..8e110954a3 100644
--- a/src/applications/almanac/editor/AlmanacDeviceEditor.php
+++ b/src/applications/almanac/editor/AlmanacDeviceEditor.php
@@ -204,7 +204,7 @@ final class AlmanacDeviceEditor
                 $xaction);
               $errors[] = $error;
             } else {
-              // TODO: Validate addresses, but IPv6 addresses are not trival
+              // TODO: Validate addresses, but IPv6 addresses are not trivial
               // to validate.
             }
 
diff --git a/src/applications/audit/view/PhabricatorAuditListView.php b/src/applications/audit/view/PhabricatorAuditListView.php
index f9399e66ab..cb9fecac3a 100644
--- a/src/applications/audit/view/PhabricatorAuditListView.php
+++ b/src/applications/audit/view/PhabricatorAuditListView.php
@@ -63,7 +63,7 @@ final class PhabricatorAuditListView extends AphrontView {
       return $summary;
     }
 
-    // No summary, so either this is still impoting or just has an empty
+    // No summary, so either this is still importing or just has an empty
     // commit message.
 
     if (!$commit->isImported()) {
diff --git a/src/applications/auth/controller/PhabricatorAuthController.php b/src/applications/auth/controller/PhabricatorAuthController.php
index c5b3c1ce99..0ed86e3056 100644
--- a/src/applications/auth/controller/PhabricatorAuthController.php
+++ b/src/applications/auth/controller/PhabricatorAuthController.php
@@ -134,7 +134,7 @@ abstract class PhabricatorAuthController extends PhabricatorController {
     // checks later on to make sure this account is valid for the intended
     // operation. This requires edit permission for completeness and consistency
     // but it won't actually be meaningfully checked because we're using the
-    // ominpotent user.
+    // omnipotent user.
 
     $account = id(new PhabricatorExternalAccountQuery())
       ->setViewer(PhabricatorUser::getOmnipotentUser())
diff --git a/src/applications/auth/controller/PhabricatorAuthRegisterController.php b/src/applications/auth/controller/PhabricatorAuthRegisterController.php
index a68e12c801..5a792dc864 100644
--- a/src/applications/auth/controller/PhabricatorAuthRegisterController.php
+++ b/src/applications/auth/controller/PhabricatorAuthRegisterController.php
@@ -79,7 +79,7 @@ final class PhabricatorAuthRegisterController
     }
 
     if ($default_email !== null) {
-      // We should bypass policy here becase e.g. limiting an application use
+      // We should bypass policy here because e.g. limiting an application use
       // to a subset of users should not allow the others to overwrite
       // configured application emails.
       $application_email = id(new PhabricatorMetaMTAApplicationEmailQuery())
diff --git a/src/applications/auth/editor/PhabricatorAuthSSHKeyEditor.php b/src/applications/auth/editor/PhabricatorAuthSSHKeyEditor.php
index 0962b7b56a..1c40af5795 100644
--- a/src/applications/auth/editor/PhabricatorAuthSSHKeyEditor.php
+++ b/src/applications/auth/editor/PhabricatorAuthSSHKeyEditor.php
@@ -226,7 +226,7 @@ final class PhabricatorAuthSSHKeyEditor
       ->addHeader('Thread-Topic', $phid);
 
     // The primary value of this mail is alerting users to account compromises,
-    // so force delivery. In particular, this mail should still be delievered
+    // so force delivery. In particular, this mail should still be delivered
     // even if "self mail" is disabled.
     $mail->setForceDelivery(true);
 
diff --git a/src/applications/auth/factor/__tests__/PhabricatorAuthInviteTestCase.php b/src/applications/auth/factor/__tests__/PhabricatorAuthInviteTestCase.php
index 3906c5eabe..ce9151bcef 100644
--- a/src/applications/auth/factor/__tests__/PhabricatorAuthInviteTestCase.php
+++ b/src/applications/auth/factor/__tests__/PhabricatorAuthInviteTestCase.php
@@ -51,7 +51,7 @@ final class PhabricatorAuthInviteTestCase extends PhabricatorTestCase {
       $caught = $ex;
     }
 
-    // This first time should accept the invite and verify the addresss.
+    // This first time should accept the invite and verify the address.
     $this->assertTrue(
       ($caught instanceof PhabricatorAuthInviteRegisteredException));
 
diff --git a/src/applications/auth/management/PhabricatorAuthManagementRevokeWorkflow.php b/src/applications/auth/management/PhabricatorAuthManagementRevokeWorkflow.php
index bb94cfca62..758761b2ca 100644
--- a/src/applications/auth/management/PhabricatorAuthManagementRevokeWorkflow.php
+++ b/src/applications/auth/management/PhabricatorAuthManagementRevokeWorkflow.php
@@ -74,7 +74,7 @@ final class PhabricatorAuthManagementRevokeWorkflow
     if (!strlen($from) && !$is_everywhere) {
       throw new PhutilArgumentUsageException(
         pht(
-          'Specify the target to revoke credentals from with "--from" or '.
+          'Specify the target to revoke credentials from with "--from" or '.
           'specify "--everywhere".'));
     } else if (strlen($from) && $is_everywhere) {
       throw new PhutilArgumentUsageException(
diff --git a/src/applications/auth/provider/PhabricatorOAuth1AuthProvider.php b/src/applications/auth/provider/PhabricatorOAuth1AuthProvider.php
index 530bf30583..c62983de2f 100644
--- a/src/applications/auth/provider/PhabricatorOAuth1AuthProvider.php
+++ b/src/applications/auth/provider/PhabricatorOAuth1AuthProvider.php
@@ -270,7 +270,7 @@ abstract class PhabricatorOAuth1AuthProvider
   }
 
   private function getHandshakeTokenKeyFromClientCode($client_code) {
-    // NOTE: This is very slightly coersive since the TemporaryToken table
+    // NOTE: This is very slightly coercive since the TemporaryToken table
     // expects an "objectPHID" as an identifier, but nothing about the storage
     // is bound to PHIDs.
 
diff --git a/src/applications/base/PhabricatorApplication.php b/src/applications/base/PhabricatorApplication.php
index 72b577771f..c25612408c 100644
--- a/src/applications/base/PhabricatorApplication.php
+++ b/src/applications/base/PhabricatorApplication.php
@@ -108,7 +108,7 @@ abstract class PhabricatorApplication
    *
    * Launchable applications can be pinned to the home page, and show up in the
    * "Launcher" view of the Applications application. Making an application
-   * unlauncahble prevents pinning and hides it from this view.
+   * unlaunchable prevents pinning and hides it from this view.
    *
    * Usually, an application should be marked unlaunchable if:
    *
diff --git a/src/applications/base/controller/PhabricatorController.php b/src/applications/base/controller/PhabricatorController.php
index 1ecfdaf557..f923eb7b73 100644
--- a/src/applications/base/controller/PhabricatorController.php
+++ b/src/applications/base/controller/PhabricatorController.php
@@ -552,7 +552,7 @@ abstract class PhabricatorController extends AphrontController {
 
 
   public function buildApplicationCrumbsForEditEngine() {
-    // TODO: This is kind of gross, I'm bascially just making this public so
+    // TODO: This is kind of gross, I'm basically just making this public so
     // I can use it in EditEngine. We could do this without making it public
     // by using controller delegation, or make it properly public.
     return $this->buildApplicationCrumbs();
diff --git a/src/applications/calendar/editor/PhabricatorCalendarImportEditEngine.php b/src/applications/calendar/editor/PhabricatorCalendarImportEditEngine.php
index c3dc29bb51..90b4962ca9 100644
--- a/src/applications/calendar/editor/PhabricatorCalendarImportEditEngine.php
+++ b/src/applications/calendar/editor/PhabricatorCalendarImportEditEngine.php
@@ -134,7 +134,7 @@ final class PhabricatorCalendarImportEditEngine
       $fields[] = id(new PhabricatorSelectEditField())
         ->setKey('frequency')
         ->setLabel(pht('Update Automatically'))
-        ->setDescription(pht('Configure an automatic update frequncy.'))
+        ->setDescription(pht('Configure an automatic update frequency.'))
         ->setTransactionType(
           PhabricatorCalendarImportFrequencyTransaction::TRANSACTIONTYPE)
         ->setConduitDescription(pht('Set the automatic update frequency.'))
diff --git a/src/applications/calendar/icon/PhabricatorCalendarIconSet.php b/src/applications/calendar/icon/PhabricatorCalendarIconSet.php
index 22d5b0e4f5..ab6d2126ed 100644
--- a/src/applications/calendar/icon/PhabricatorCalendarIconSet.php
+++ b/src/applications/calendar/icon/PhabricatorCalendarIconSet.php
@@ -16,7 +16,7 @@ final class PhabricatorCalendarIconSet
       'fa-plane' => pht('Travel'),
       'fa-plus-square' => pht('Health / Appointment'),
 
-      'fa-rocket' => pht('Sabatical / Leave'),
+      'fa-rocket' => pht('Sabbatical / Leave'),
       'fa-home' => pht('Working From Home'),
       'fa-tree' => pht('Holiday'),
       'fa-gamepad' => pht('Staycation'),
diff --git a/src/applications/calendar/util/CalendarTimeUtil.php b/src/applications/calendar/util/CalendarTimeUtil.php
index 781f6adf7a..d3276654e4 100644
--- a/src/applications/calendar/util/CalendarTimeUtil.php
+++ b/src/applications/calendar/util/CalendarTimeUtil.php
@@ -8,7 +8,7 @@
  * calendar views - one for the conpherence calendar widget and one for the
  * user profile calendar view. These have slight differences such as
  * conpherence showing both a three day "today 'til 2 days from now" *and*
- * a Sunday -> Saturday list, whilest the profile view shows a more simple
+ * a Sunday -> Saturday list, whilst the profile view shows a more simple
  * seven day rolling list of events.
  */
 final class CalendarTimeUtil extends Phobject {
diff --git a/src/applications/calendar/xaction/PhabricatorCalendarEventFrequencyTransaction.php b/src/applications/calendar/xaction/PhabricatorCalendarEventFrequencyTransaction.php
index 6fb8ae8f31..69f2e9e8a6 100644
--- a/src/applications/calendar/xaction/PhabricatorCalendarEventFrequencyTransaction.php
+++ b/src/applications/calendar/xaction/PhabricatorCalendarEventFrequencyTransaction.php
@@ -23,7 +23,7 @@ final class PhabricatorCalendarEventFrequencyTransaction
     // month, it means "the 30th of every month" as far as the RRULE is
     // concerned. Such an event will not occur on months with fewer days.
 
-    // This is surprising, and proably not what the user wants. Instead,
+    // This is surprising, and probably not what the user wants. Instead,
     // schedule these events relative to the end of the month: on the "-1st",
     // "-2nd" or "-3rd" day of the month. For example, a monthly event on
     // the 31st of a 31-day month translates to "every month, on the last
@@ -66,7 +66,7 @@ final class PhabricatorCalendarEventFrequencyTransaction
       if (!isset($valid[$value])) {
         $errors[] = $this->newInvalidError(
           pht(
-            'Event frequency "%s" is not valid. Valid frequences are: %s.',
+            'Event frequency "%s" is not valid. Valid frequencies are: %s.',
             $value,
             implode(', ', $valid)),
           $xaction);
diff --git a/src/applications/calendar/xaction/PhabricatorCalendarEventInviteTransaction.php b/src/applications/calendar/xaction/PhabricatorCalendarEventInviteTransaction.php
index f1da2c1d77..841a066320 100644
--- a/src/applications/calendar/xaction/PhabricatorCalendarEventInviteTransaction.php
+++ b/src/applications/calendar/xaction/PhabricatorCalendarEventInviteTransaction.php
@@ -132,7 +132,7 @@ final class PhabricatorCalendarEventInviteTransaction
         $this->renderHandleList($rem));
     } else {
       return pht(
-        '%s invited %s attendee(s): %s; uninvinted %s attendee(s): %s.',
+        '%s invited %s attendee(s): %s; uninvited %s attendee(s): %s.',
         $this->renderAuthor(),
         phutil_count($add),
         $this->renderHandleList($add),
@@ -161,7 +161,7 @@ final class PhabricatorCalendarEventInviteTransaction
     } else {
       return pht(
         '%s updated the invite list for %s, invited %s: %s; '.
-        'uninvinted %s: %s.',
+        'uninvited %s: %s.',
         $this->renderAuthor(),
         $this->renderObject(),
         phutil_count($add),
diff --git a/src/applications/calendar/xaction/PhabricatorCalendarImportFrequencyTransaction.php b/src/applications/calendar/xaction/PhabricatorCalendarImportFrequencyTransaction.php
index 177adbbf0b..9fc68a3497 100644
--- a/src/applications/calendar/xaction/PhabricatorCalendarImportFrequencyTransaction.php
+++ b/src/applications/calendar/xaction/PhabricatorCalendarImportFrequencyTransaction.php
@@ -32,7 +32,7 @@ final class PhabricatorCalendarImportFrequencyTransaction
       if (!isset($valid[$value])) {
         $errors[] = $this->newInvalidError(
           pht(
-            'Import frequency "%s" is not valid. Valid frequences are: %s.',
+            'Import frequency "%s" is not valid. Valid frequencies are: %s.',
             $value,
             implode(', ', $valid)),
           $xaction);
diff --git a/src/applications/config/check/PhabricatorStorageSetupCheck.php b/src/applications/config/check/PhabricatorStorageSetupCheck.php
index 09cecfe6d0..cc74cce2ea 100644
--- a/src/applications/config/check/PhabricatorStorageSetupCheck.php
+++ b/src/applications/config/check/PhabricatorStorageSetupCheck.php
@@ -85,7 +85,7 @@ final class PhabricatorStorageSetupCheck extends PhabricatorSetupCheck {
           'be buffered into memory before being written to permanent '.
           'storage. Phabricator needs memory available to store these '.
           'chunks while they are uploaded, but PHP is currently configured '.
-          'to severly limit the available memory.'.
+          'to severely limit the available memory.'.
           "\n\n".
           'PHP processes currently have very little free memory available '.
           '(%s). To work well, processes should have at least %s.'.
diff --git a/src/applications/conpherence/query/ConpherenceThreadQuery.php b/src/applications/conpherence/query/ConpherenceThreadQuery.php
index 1bb96537dc..5cd6489d65 100644
--- a/src/applications/conpherence/query/ConpherenceThreadQuery.php
+++ b/src/applications/conpherence/query/ConpherenceThreadQuery.php
@@ -290,7 +290,7 @@ final class ConpherenceThreadQuery
       ->withObjectPHIDs(array_keys($conpherences))
       ->needHandles(true);
 
-    // We have to flip these for the underyling query class. The semantics of
+    // We have to flip these for the underlying query class. The semantics of
     // paging are tricky business.
     if ($this->afterTransactionID) {
       $query->setBeforeID($this->afterTransactionID);
diff --git a/src/applications/dashboard/customfield/PhabricatorDashboardPanelTabsCustomField.php b/src/applications/dashboard/customfield/PhabricatorDashboardPanelTabsCustomField.php
index 96566db54a..07bbb67ae2 100644
--- a/src/applications/dashboard/customfield/PhabricatorDashboardPanelTabsCustomField.php
+++ b/src/applications/dashboard/customfield/PhabricatorDashboardPanelTabsCustomField.php
@@ -78,7 +78,7 @@ final class PhabricatorDashboardPanelTabsCustomField
 
   public function renderEditControl(array $handles) {
     // NOTE: This includes archived panels so we don't mutate the tabs
-    // when saving a tab panel that includes archied panels. This whole UI is
+    // when saving a tab panel that includes archived panels. This whole UI is
     // hopefully temporary anyway.
 
     $value = $this->getFieldValue();
diff --git a/src/applications/differential/controller/DifferentialController.php b/src/applications/differential/controller/DifferentialController.php
index fb8a6249a7..1860a07745 100644
--- a/src/applications/differential/controller/DifferentialController.php
+++ b/src/applications/differential/controller/DifferentialController.php
@@ -217,7 +217,7 @@ abstract class DifferentialController extends PhabricatorController {
       // by default and let the user toggle the rest. With modern messages,
       // we can send the user to the Harbormaster detail page. Just show
       // "a lot" of messages in legacy cases to try to strike a balance
-      // between implementation simplicitly and compatibility.
+      // between implementation simplicity and compatibility.
       $legacy_messages = array_slice($legacy_messages, 0, 100);
 
       $messages = array();
diff --git a/src/applications/differential/customfield/DifferentialHarbormasterField.php b/src/applications/differential/customfield/DifferentialHarbormasterField.php
index c9b573bda2..cc13be3aa7 100644
--- a/src/applications/differential/customfield/DifferentialHarbormasterField.php
+++ b/src/applications/differential/customfield/DifferentialHarbormasterField.php
@@ -45,7 +45,7 @@ abstract class DifferentialHarbormasterField
         // by default and let the user toggle the rest. With modern messages,
         // we can send the user to the Harbormaster detail page. Just show
         // "a lot" of messages in legacy cases to try to strike a balance
-        // between implementation simplicitly and compatibility.
+        // between implementation simplicity and compatibility.
         $legacy_messages = array_slice($legacy_messages, 0, 100);
 
         foreach ($legacy_messages as $message) {
diff --git a/src/applications/differential/editor/DifferentialDiffEditor.php b/src/applications/differential/editor/DifferentialDiffEditor.php
index 0058485e63..261f6f1598 100644
--- a/src/applications/differential/editor/DifferentialDiffEditor.php
+++ b/src/applications/differential/editor/DifferentialDiffEditor.php
@@ -106,7 +106,7 @@ final class DifferentialDiffEditor
    * We run Herald as part of transaction validation because Herald can
    * block diff creation for Differential diffs. Its important to do this
    * separately so no Herald logs are saved; these logs could expose
-   * information the Herald rules are inteneded to block.
+   * information the Herald rules are intended to block.
    */
   protected function validateTransaction(
     PhabricatorLiskDAO $object,
diff --git a/src/applications/differential/parser/DifferentialChangesetParser.php b/src/applications/differential/parser/DifferentialChangesetParser.php
index 88303cf420..1eeb4c8452 100644
--- a/src/applications/differential/parser/DifferentialChangesetParser.php
+++ b/src/applications/differential/parser/DifferentialChangesetParser.php
@@ -699,7 +699,7 @@ final class DifferentialChangesetParser extends Phobject {
     $hunk_parser->parseHunksForLineData($changeset->getHunks());
 
     // Depending on the whitespace mode, we may need to compute a different
-    // set of changes than the set of changes in the hunk data (specificaly,
+    // set of changes than the set of changes in the hunk data (specifically,
     // we might want to consider changed lines which have only whitespace
     // changes as unchanged).
     if ($ignore_all) {
@@ -1192,11 +1192,11 @@ final class DifferentialChangesetParser extends Phobject {
    * Mask - compute the actual lines that need to be shown (because they
    * are near changes lines, near inline comments, or the request has
    * explicitly asked for them, i.e. resulting from the user clicking
-   * "show more"). The $mask returned is a sparesely populated dictionary
+   * "show more"). The $mask returned is a sparsely populated dictionary
    * of $visible_line_number => true.
    *
    * Depths - compute how indented any given line is. The $depths returned
-   * is a sparesely populated dictionary of $visible_line_number => $depth.
+   * is a sparsely populated dictionary of $visible_line_number => $depth.
    *
    * This function also has the side effect of modifying member variable
    * new such that tabs are normalized to spaces for each line of the diff.
diff --git a/src/applications/differential/storage/DifferentialModernHunk.php b/src/applications/differential/storage/DifferentialModernHunk.php
index cde6f29329..c3675c8adf 100644
--- a/src/applications/differential/storage/DifferentialModernHunk.php
+++ b/src/applications/differential/storage/DifferentialModernHunk.php
@@ -128,7 +128,7 @@ final class DifferentialModernHunk extends DifferentialHunk {
     $this->setData($file->getPHID());
 
     // NOTE: Because hunks don't have a PHID and we just load hunk data with
-    // the ominipotent viewer, we do not need to attach the file to anything.
+    // the omnipotent viewer, we do not need to attach the file to anything.
 
     $result = $this->save();
 
diff --git a/src/applications/differential/storage/DifferentialRevision.php b/src/applications/differential/storage/DifferentialRevision.php
index 7bcd178a41..e29db36b14 100644
--- a/src/applications/differential/storage/DifferentialRevision.php
+++ b/src/applications/differential/storage/DifferentialRevision.php
@@ -442,7 +442,7 @@ final class DifferentialRevision extends DifferentialDAO
 
     // For each path which the viewer owns a package for, find other packages
     // which that authority can be used to force-accept. Once we find a way to
-    // force-accept a package, we don't need to keep loooking.
+    // force-accept a package, we don't need to keep looking.
     $has_control = array();
     foreach ($force_map as $path => $spec) {
       $path_fragments = PhabricatorOwnersPackage::splitPath($path);
@@ -891,7 +891,7 @@ final class DifferentialRevision extends DifferentialDAO
         self::TABLE_COMMIT,
         $this->getID());
 
-      // we have to do paths a little differentally as they do not have
+      // we have to do paths a little differently as they do not have
       // an id or phid column for delete() to act on
       $dummy_path = new DifferentialAffectedPath();
       queryfx(
diff --git a/src/applications/differential/typeahead/DifferentialResponsibleViewerFunctionDatasource.php b/src/applications/differential/typeahead/DifferentialResponsibleViewerFunctionDatasource.php
index 51eb2beb9a..d23ecff47d 100644
--- a/src/applications/differential/typeahead/DifferentialResponsibleViewerFunctionDatasource.php
+++ b/src/applications/differential/typeahead/DifferentialResponsibleViewerFunctionDatasource.php
@@ -22,7 +22,7 @@ final class DifferentialResponsibleViewerFunctionDatasource
         'summary' => pht('Use the current viewing user.'),
         'description' => pht(
           'Show revisions the current viewer is responsible for. This '.
-          'function inclues revisions the viewer is responsible for through '.
+          'function includes revisions the viewer is responsible for through '.
           'membership in projects and packages.'),
       ),
     );
diff --git a/src/applications/diffusion/controller/DiffusionCommitController.php b/src/applications/diffusion/controller/DiffusionCommitController.php
index 38c94e8a92..d7734856f0 100644
--- a/src/applications/diffusion/controller/DiffusionCommitController.php
+++ b/src/applications/diffusion/controller/DiffusionCommitController.php
@@ -479,7 +479,7 @@ final class DiffusionCommitController extends DiffusionController {
     // chains of events). This should be rare, but does not indicate a bug
     // or data issue.
 
-    // NOTE: We never query push logs in SVN because the commiter is always
+    // NOTE: We never query push logs in SVN because the committer is always
     // the pusher and the commit time is always the push time; the push log
     // is redundant and we save a query by skipping it.
 
diff --git a/src/applications/diffusion/editor/DiffusionRepositoryEditEngine.php b/src/applications/diffusion/editor/DiffusionRepositoryEditEngine.php
index 2459c8c559..b3baafbd06 100644
--- a/src/applications/diffusion/editor/DiffusionRepositoryEditEngine.php
+++ b/src/applications/diffusion/editor/DiffusionRepositoryEditEngine.php
@@ -338,7 +338,7 @@ final class DiffusionRepositoryEditEngine
         ->setIsCopyable(true)
         ->setDescription(pht('Track only these branches.'))
         ->setConduitDescription(pht('Set the tracked branches.'))
-        ->setConduitTypeDescription(pht('New tracked branchs.'))
+        ->setConduitTypeDescription(pht('New tracked branches.'))
         ->setValue($track_value),
       id(new PhabricatorTextAreaEditField())
         ->setIsStringList(true)
@@ -349,7 +349,7 @@ final class DiffusionRepositoryEditEngine
         ->setIsCopyable(true)
         ->setDescription(pht('Autoclose commits on only these branches.'))
         ->setConduitDescription(pht('Set the autoclose branches.'))
-        ->setConduitTypeDescription(pht('New default tracked branchs.'))
+        ->setConduitTypeDescription(pht('New default tracked branches.'))
         ->setValue($autoclose_value),
       id(new PhabricatorTextEditField())
         ->setKey('importOnly')
@@ -396,7 +396,7 @@ final class DiffusionRepositoryEditEngine
         ->setConduitDescription(
           pht('Change symbol languages for this repository.'))
         ->setConduitTypeDescription(
-          pht('New symbol langauges.'))
+          pht('New symbol languages.'))
         ->setValue($object->getSymbolLanguages()),
       id(new PhabricatorDatasourceEditField())
         ->setKey('symbolRepositoryPHIDs')
diff --git a/src/applications/diffusion/engine/DiffusionCommitHookEngine.php b/src/applications/diffusion/engine/DiffusionCommitHookEngine.php
index 21df37d87d..7fe45834b3 100644
--- a/src/applications/diffusion/engine/DiffusionCommitHookEngine.php
+++ b/src/applications/diffusion/engine/DiffusionCommitHookEngine.php
@@ -1230,7 +1230,7 @@ final class DiffusionCommitHookEngine extends Phobject {
     // also look at author data (do the commits come from multiple different
     // authors?) and commit date data (is the oldest commit more than 48 hours
     // old), but we don't have immediate access to those and this simple
-    // heruistic might be good enough.
+    // heuristic might be good enough.
 
     $commit_count = 0;
     $type_commit = PhabricatorRepositoryPushLog::REFTYPE_COMMIT;
diff --git a/src/applications/diffusion/gitlfs/DiffusionGitLFSAuthenticateWorkflow.php b/src/applications/diffusion/gitlfs/DiffusionGitLFSAuthenticateWorkflow.php
index aca7367f8c..41c009455d 100644
--- a/src/applications/diffusion/gitlfs/DiffusionGitLFSAuthenticateWorkflow.php
+++ b/src/applications/diffusion/gitlfs/DiffusionGitLFSAuthenticateWorkflow.php
@@ -80,7 +80,7 @@ final class DiffusionGitLFSAuthenticateWorkflow
 
     $lfs_uri = $repository->getGitLFSURI('info/lfs');
 
-    // Generate a temporary token to allow the user to acces LFS over HTTP.
+    // Generate a temporary token to allow the user to access LFS over HTTP.
     // This works even if normal HTTP repository operations are not available
     // on this host, and does not require the user to have a VCS password.
 
diff --git a/src/applications/diffusion/herald/HeraldCommitAdapter.php b/src/applications/diffusion/herald/HeraldCommitAdapter.php
index 4687028418..bfcd8e3ccb 100644
--- a/src/applications/diffusion/herald/HeraldCommitAdapter.php
+++ b/src/applications/diffusion/herald/HeraldCommitAdapter.php
@@ -112,7 +112,7 @@ final class HeraldCommitAdapter
     $phids[] = $repository_phid;
 
     // NOTE: This is projects for the repository, not for the commit. When
-    // Herald evalutes, commits normally can not have any project tags yet.
+    // Herald evaluates, commits normally can not have any project tags yet.
     $repository_project_phids = PhabricatorEdgeQuery::loadDestinationPHIDs(
       $repository_phid,
       $project_type);
diff --git a/src/applications/diffusion/protocol/DiffusionCommandEngine.php b/src/applications/diffusion/protocol/DiffusionCommandEngine.php
index b787745a02..53a086db33 100644
--- a/src/applications/diffusion/protocol/DiffusionCommandEngine.php
+++ b/src/applications/diffusion/protocol/DiffusionCommandEngine.php
@@ -178,7 +178,7 @@ abstract class DiffusionCommandEngine extends Phobject {
       if (!$device) {
         throw new Exception(
           pht(
-            'Attempting to build a reposiory command (for repository "%s") '.
+            'Attempting to build a repository command (for repository "%s") '.
             'as device, but this host ("%s") is not configured as a cluster '.
             'device.',
             $repository->getDisplayName(),
diff --git a/src/applications/diffusion/protocol/DiffusionMercurialWireProtocol.php b/src/applications/diffusion/protocol/DiffusionMercurialWireProtocol.php
index 32a73867e5..251935a6dc 100644
--- a/src/applications/diffusion/protocol/DiffusionMercurialWireProtocol.php
+++ b/src/applications/diffusion/protocol/DiffusionMercurialWireProtocol.php
@@ -101,7 +101,7 @@ final class DiffusionMercurialWireProtocol extends Phobject {
 
   /** If the server version is running 3.4+ it will respond
    * with 'bundle2' capability in the format of "bundle2=(url-encoding)".
-   * Until we maange to properly package up bundles to send back we
+   * Until we manage to properly package up bundles to send back we
    * disallow the client from knowing we speak bundle2 by removing it
    * from the capabilities listing.
    *
diff --git a/src/applications/diffusion/protocol/DiffusionRepositoryClusterEngine.php b/src/applications/diffusion/protocol/DiffusionRepositoryClusterEngine.php
index e822bb76b0..35bb4d4acf 100644
--- a/src/applications/diffusion/protocol/DiffusionRepositoryClusterEngine.php
+++ b/src/applications/diffusion/protocol/DiffusionRepositoryClusterEngine.php
@@ -231,7 +231,7 @@ final class DiffusionRepositoryClusterEngine extends Phobject {
       // no way to tell which one has the "right" data. If we pick wrong, we
       // might erase some or all of the data in the repository.
 
-      // Since this is dangeorus, we refuse to guess unless there is only one
+      // Since this is dangerous, we refuse to guess unless there is only one
       // device. If we're the only device in the group, we obviously must be
       // a leader.
 
@@ -252,7 +252,7 @@ final class DiffusionRepositoryClusterEngine extends Phobject {
             'Repository "%s" exists on more than one device, but no device '.
             'has any repository version information. Phabricator can not '.
             'guess which copy of the existing data is authoritative. Promote '.
-            'a device or see "Ambigous Leaders" in the documentation.',
+            'a device or see "Ambiguous Leaders" in the documentation.',
             $repository->getDisplayName()));
       }
 
diff --git a/src/applications/diffusion/query/DiffusionCachedResolveRefsQuery.php b/src/applications/diffusion/query/DiffusionCachedResolveRefsQuery.php
index 22ff1d61f0..0217c97d58 100644
--- a/src/applications/diffusion/query/DiffusionCachedResolveRefsQuery.php
+++ b/src/applications/diffusion/query/DiffusionCachedResolveRefsQuery.php
@@ -9,7 +9,7 @@
  * low-level query can.
  *
  * This class can resolve the most common refs (commits, branches, tags) and
- * can do so cheapy (by examining the database, without needing to make calls
+ * can do so cheaply (by examining the database, without needing to make calls
  * to the VCS or the service host).
  */
 final class DiffusionCachedResolveRefsQuery
diff --git a/src/applications/diffusion/query/DiffusionCommitQuery.php b/src/applications/diffusion/query/DiffusionCommitQuery.php
index 1521f5770c..da8937910e 100644
--- a/src/applications/diffusion/query/DiffusionCommitQuery.php
+++ b/src/applications/diffusion/query/DiffusionCommitQuery.php
@@ -69,7 +69,7 @@ final class DiffusionCommitQuery
 
   /**
    * Look up commits in a specific repository. Prefer
-   * @{method:withRepositoryIDs}; the underyling table is keyed by ID such
+   * @{method:withRepositoryIDs}; the underlying table is keyed by ID such
    * that this method requires a separate initial query to map PHID to ID.
    */
   public function withRepositoryPHIDs(array $phids) {
diff --git a/src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php b/src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php
index ff1cc9f3c5..30ec8d9f6d 100644
--- a/src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php
+++ b/src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php
@@ -53,7 +53,7 @@ final class DiffusionCommitConcernTransaction
     }
 
     // Even if you've already raised a concern, you can raise again as long
-    // as the author requsted you verify.
+    // as the author requested you verify.
     $state_verify = PhabricatorAuditCommitStatusConstants::NEEDS_VERIFICATION;
 
     if ($this->isViewerFullyRejected($object, $viewer)) {
diff --git a/src/applications/diffusion/xaction/DiffusionCommitVerifyTransaction.php b/src/applications/diffusion/xaction/DiffusionCommitVerifyTransaction.php
index f9b3fc5161..a44261f7a4 100644
--- a/src/applications/diffusion/xaction/DiffusionCommitVerifyTransaction.php
+++ b/src/applications/diffusion/xaction/DiffusionCommitVerifyTransaction.php
@@ -53,7 +53,7 @@ final class DiffusionCommitVerifyTransaction
       throw new Exception(
         pht(
           'You can not request verification of this commit because no '.
-          'auditors have raised conerns with it.'));
+          'auditors have raised concerns with it.'));
     }
   }
 
diff --git a/src/applications/drydock/blueprint/DrydockBlueprintImplementation.php b/src/applications/drydock/blueprint/DrydockBlueprintImplementation.php
index 76a81d7ef1..a050a859e3 100644
--- a/src/applications/drydock/blueprint/DrydockBlueprintImplementation.php
+++ b/src/applications/drydock/blueprint/DrydockBlueprintImplementation.php
@@ -64,7 +64,7 @@ abstract class DrydockBlueprintImplementation extends Phobject {
    * is a coarse compatibility check between a lease and a resource.
    *
    * @param DrydockBlueprint Concrete blueprint to allocate for.
-   * @param DrydockResource Candidiate resource to allocate the lease on.
+   * @param DrydockResource Candidate resource to allocate the lease on.
    * @param DrydockLease Pending lease that wants to allocate here.
    * @return bool True if the resource and lease are compatible.
    * @task lease
@@ -76,7 +76,7 @@ abstract class DrydockBlueprintImplementation extends Phobject {
 
 
   /**
-   * Acquire a lease. Allows resources to peform setup as leases are brought
+   * Acquire a lease. Allows resources to perform setup as leases are brought
    * online.
    *
    * If acquisition fails, throw an exception.
diff --git a/src/applications/drydock/storage/DrydockAuthorization.php b/src/applications/drydock/storage/DrydockAuthorization.php
index cfd186c9d3..32e694918c 100644
--- a/src/applications/drydock/storage/DrydockAuthorization.php
+++ b/src/applications/drydock/storage/DrydockAuthorization.php
@@ -100,7 +100,7 @@ final class DrydockAuthorization extends DrydockDAO
   }
 
   /**
-   * Apply external authorization effects after a user chagnes the value of a
+   * Apply external authorization effects after a user changes the value of a
    * blueprint selector control an object.
    *
    * @param PhabricatorUser User applying the change.
diff --git a/src/applications/drydock/worker/DrydockLeaseUpdateWorker.php b/src/applications/drydock/worker/DrydockLeaseUpdateWorker.php
index a41e57fbf8..d73506f286 100644
--- a/src/applications/drydock/worker/DrydockLeaseUpdateWorker.php
+++ b/src/applications/drydock/worker/DrydockLeaseUpdateWorker.php
@@ -241,7 +241,7 @@ final class DrydockLeaseUpdateWorker extends DrydockWorker {
       // NOTE: We have not acquired the lease yet, so it is possible that the
       // resource we just built will be snatched up by some other lease before
       // we can acquire it. This is not problematic: we'll retry a little later
-      // and should suceed eventually.
+      // and should succeed eventually.
     }
 
     $resources = $this->rankResources($resources, $lease);
@@ -261,7 +261,7 @@ final class DrydockLeaseUpdateWorker extends DrydockWorker {
     if (!$allocated) {
       throw new PhutilAggregateException(
         pht(
-          'Unable to acquire lease "%s" on any resouce.',
+          'Unable to acquire lease "%s" on any resource.',
           $lease->getPHID()),
         $exceptions);
     }
@@ -725,7 +725,7 @@ final class DrydockLeaseUpdateWorker extends DrydockWorker {
     // performed the read above and now, the resource might have closed, so
     // we may activate leases on dead resources. At least for now, this seems
     // fine: a resource dying right before we activate a lease on it should not
-    // be distinguisahble from a resource dying right after we activate a lease
+    // be distinguishable from a resource dying right after we activate a lease
     // on it. We end up with an active lease on a dead resource either way, and
     // can not prevent resources dying from lightning strikes.
 
diff --git a/src/applications/drydock/worker/DrydockWorker.php b/src/applications/drydock/worker/DrydockWorker.php
index f167fef5d0..443780680d 100644
--- a/src/applications/drydock/worker/DrydockWorker.php
+++ b/src/applications/drydock/worker/DrydockWorker.php
@@ -199,7 +199,7 @@ abstract class DrydockWorker extends PhabricatorWorker {
     $viewer = $this->getViewer();
 
     // Don't reclaim a resource if it has been updated recently. If two
-    // leases are fighting, we don't want them to keep reclaming resources
+    // leases are fighting, we don't want them to keep reclaiming resources
     // from one another forever without making progress, so make resources
     // immune to reclamation for a little while after they activate or update.
 
diff --git a/src/applications/files/controller/PhabricatorFileDataController.php b/src/applications/files/controller/PhabricatorFileDataController.php
index c8bfcc488a..da438730cd 100644
--- a/src/applications/files/controller/PhabricatorFileDataController.php
+++ b/src/applications/files/controller/PhabricatorFileDataController.php
@@ -119,7 +119,7 @@ final class PhabricatorFileDataController extends PhabricatorFileController {
     // make this logic simpler and more consistent.
 
     // Beyond making the policy check itself more consistent, this also makes
-    // sure we're consitent about returning HTTP 404 on bad requests instead
+    // sure we're consistent about returning HTTP 404 on bad requests instead
     // of serving HTTP 200 with a login page, which can mislead some clients.
 
     $viewer = PhabricatorUser::getOmnipotentUser();
diff --git a/src/applications/files/management/PhabricatorFilesManagementCatWorkflow.php b/src/applications/files/management/PhabricatorFilesManagementCatWorkflow.php
index 70d5eca5a2..5bf40a057d 100644
--- a/src/applications/files/management/PhabricatorFilesManagementCatWorkflow.php
+++ b/src/applications/files/management/PhabricatorFilesManagementCatWorkflow.php
@@ -24,7 +24,7 @@ final class PhabricatorFilesManagementCatWorkflow
             'help' => pht(
               'DANGEROUS. Attempt to salvage file content even if the '.
               'integrity check fails. If an adversary has tampered with '.
-              'the file, the conent may be unsafe.'),
+              'the file, the content may be unsafe.'),
           ),
           array(
             'name'      => 'names',
diff --git a/src/applications/fund/editor/FundInitiativeEditEngine.php b/src/applications/fund/editor/FundInitiativeEditEngine.php
index 201a814a0c..824f4e009f 100644
--- a/src/applications/fund/editor/FundInitiativeEditEngine.php
+++ b/src/applications/fund/editor/FundInitiativeEditEngine.php
@@ -50,7 +50,7 @@ final class FundInitiativeEditEngine
   }
 
   protected function getObjectName() {
-    return pht('Initivative');
+    return pht('Initiative');
   }
 
   protected function getObjectCreateCancelURI($object) {
diff --git a/src/applications/harbormaster/conduit/HarbormasterSendMessageConduitAPIMethod.php b/src/applications/harbormaster/conduit/HarbormasterSendMessageConduitAPIMethod.php
index 9c3f88d631..dbf1f18cfa 100644
--- a/src/applications/harbormaster/conduit/HarbormasterSendMessageConduitAPIMethod.php
+++ b/src/applications/harbormaster/conduit/HarbormasterSendMessageConduitAPIMethod.php
@@ -142,7 +142,7 @@ final class HarbormasterSendMessageConduitAPIMethod
       "=============\n".
       "When you send Harbormaster a message, you must include a `type`, ".
       "which describes the overall state of the build. For example, use ".
-      "`pass` to tell Harbomaster that a build completed successfully.".
+      "`pass` to tell Harbormaster that a build completed successfully.".
       "\n\n".
       "Supported message types are:".
       "\n\n".
@@ -155,7 +155,7 @@ final class HarbormasterSendMessageConduitAPIMethod
       "message, but you can also send a `work` message to report intermediate ".
       "results.\n\n".
       "To provide unit test results, pass a list of results in the `unit` ".
-      "parameter. Each result shoud be a dictionary with these keys:".
+      "parameter. Each result should be a dictionary with these keys:".
       "\n\n".
       "%s".
       "\n\n".
diff --git a/src/applications/harbormaster/storage/configuration/HarbormasterBuildStep.php b/src/applications/harbormaster/storage/configuration/HarbormasterBuildStep.php
index cdb2044cc5..b29958882c 100644
--- a/src/applications/harbormaster/storage/configuration/HarbormasterBuildStep.php
+++ b/src/applications/harbormaster/storage/configuration/HarbormasterBuildStep.php
@@ -38,7 +38,7 @@ final class HarbormasterBuildStep extends HarbormasterDAO
         // T6203/NULLABILITY
         // This should not be nullable. Current `null` values indicate steps
         // which predated editable names. These should be backfilled with
-        // default names, then the code for handling `null` shoudl be removed.
+        // default names, then the code for handling `null` should be removed.
         'name' => 'text255?',
         'stepAutoKey' => 'text32?',
       ),
diff --git a/src/applications/legalpad/controller/LegalpadDocumentSignController.php b/src/applications/legalpad/controller/LegalpadDocumentSignController.php
index 05a691e34a..f19cf43d0e 100644
--- a/src/applications/legalpad/controller/LegalpadDocumentSignController.php
+++ b/src/applications/legalpad/controller/LegalpadDocumentSignController.php
@@ -236,7 +236,7 @@ final class LegalpadDocumentSignController extends LegalpadController {
 
     // Use the last content update as the modified date. We don't want to
     // show that a document like a TOS was "updated" by an incidental change
-    // to a field like the preamble or privacy settings which does not acutally
+    // to a field like the preamble or privacy settings which does not actually
     // affect the content of the agreement.
     $content_updated = $document_body->getDateCreated();
 
diff --git a/src/applications/maniphest/controller/ManiphestReportController.php b/src/applications/maniphest/controller/ManiphestReportController.php
index 6a5b3ad50c..66aa154e8f 100644
--- a/src/applications/maniphest/controller/ManiphestReportController.php
+++ b/src/applications/maniphest/controller/ManiphestReportController.php
@@ -116,7 +116,7 @@ final class ManiphestReportController extends ManiphestController {
         case ManiphestTaskMergedIntoTransaction::TRANSACTIONTYPE:
           // NOTE: Merging a task does not generate a "status" transaction.
           // We pretend it did. Note that this is not always accurate: it is
-          // possble to merge a task which was previously closed, but this
+          // possible to merge a task which was previously closed, but this
           // fake transaction always counts a merge as a closure.
           $oldv = $default_status;
           $newv = $duplicate_status;
@@ -762,7 +762,7 @@ final class ManiphestReportController extends ManiphestController {
     // and equal distance in the past. This is so users can type "6 days" (which
     // means "6 days from now") and get the behavior of "6 days ago", rather
     // than no results (because the window epoch is in the future). This might
-    // be a little confusing because it casues "tomorrow" to mean "yesterday"
+    // be a little confusing because it causes "tomorrow" to mean "yesterday"
     // and "2022" (or whatever) to mean "ten years ago", but these inputs are
     // nonsense anyway.
 
diff --git a/src/applications/maniphest/editor/ManiphestTransactionEditor.php b/src/applications/maniphest/editor/ManiphestTransactionEditor.php
index 1b780549dc..caf70b8f3c 100644
--- a/src/applications/maniphest/editor/ManiphestTransactionEditor.php
+++ b/src/applications/maniphest/editor/ManiphestTransactionEditor.php
@@ -972,7 +972,7 @@ final class ManiphestTransactionEditor
 
     $object_phid = $object->getPHID();
 
-    // We're doing layout with the ominpotent viewer to make sure we don't
+    // We're doing layout with the omnipotent viewer to make sure we don't
     // remove positions in columns that exist, but which the actual actor
     // can't see.
     $omnipotent_viewer = PhabricatorUser::getOmnipotentUser();
diff --git a/src/applications/metamta/management/PhabricatorMailManagementUnverifyWorkflow.php b/src/applications/metamta/management/PhabricatorMailManagementUnverifyWorkflow.php
index 61861d1db0..64f4976ebe 100644
--- a/src/applications/metamta/management/PhabricatorMailManagementUnverifyWorkflow.php
+++ b/src/applications/metamta/management/PhabricatorMailManagementUnverifyWorkflow.php
@@ -63,7 +63,7 @@ final class PhabricatorMailManagementUnverifyWorkflow
         echo tsprintf(
           "%s\n",
           pht(
-            'Address "%s" (owned by "%s") is already unveriifed.',
+            'Address "%s" (owned by "%s") is already unverified.',
             $address,
             $user->getUsername()));
         continue;
diff --git a/src/applications/metamta/receiver/PhabricatorMailReceiver.php b/src/applications/metamta/receiver/PhabricatorMailReceiver.php
index 05b44f364a..12483bf5b6 100644
--- a/src/applications/metamta/receiver/PhabricatorMailReceiver.php
+++ b/src/applications/metamta/receiver/PhabricatorMailReceiver.php
@@ -255,7 +255,7 @@ abstract class PhabricatorMailReceiver extends Phobject {
 
 
   /**
-   * Reduce an email address to its canonical form. For example, an adddress
+   * Reduce an email address to its canonical form. For example, an address
    * like:
    *
    *  "Abraham Lincoln" < ALincoln@example.com >
diff --git a/src/applications/owners/storage/__tests__/PhabricatorOwnersPackageTestCase.php b/src/applications/owners/storage/__tests__/PhabricatorOwnersPackageTestCase.php
index f21f09b44e..d219abfc1e 100644
--- a/src/applications/owners/storage/__tests__/PhabricatorOwnersPackageTestCase.php
+++ b/src/applications/owners/storage/__tests__/PhabricatorOwnersPackageTestCase.php
@@ -82,7 +82,7 @@ final class PhabricatorOwnersPackageTestCase extends PhabricatorTestCase {
 
     // Now, add a more specific path to Package #1. This tests nested ownership
     // in packages with weak dominion rules. This time, Package #1 should end
-    // up back on top, with Package #2 cedeing control to its more specific
+    // up back on top, with Package #2 ceding control to its more specific
     // path.
     $rows[] = array(
       'id' => 1,
diff --git a/src/applications/passphrase/credentialtype/PassphraseCredentialType.php b/src/applications/passphrase/credentialtype/PassphraseCredentialType.php
index 1da93bcd38..60cb9bb5ae 100644
--- a/src/applications/passphrase/credentialtype/PassphraseCredentialType.php
+++ b/src/applications/passphrase/credentialtype/PassphraseCredentialType.php
@@ -104,7 +104,7 @@ abstract class PassphraseCredentialType extends Phobject {
 
 
   /**
-   * Return true if the provided credental requires a password to decrypt.
+   * Return true if the provided credential requires a password to decrypt.
    *
    * @param PhutilOpaqueEnvelope  Credential secret value.
    * @return bool                 True if the credential needs a password.
diff --git a/src/applications/passphrase/credentialtype/PassphraseSSHPrivateKeyTextCredentialType.php b/src/applications/passphrase/credentialtype/PassphraseSSHPrivateKeyTextCredentialType.php
index d573f19f58..fa686aac4c 100644
--- a/src/applications/passphrase/credentialtype/PassphraseSSHPrivateKeyTextCredentialType.php
+++ b/src/applications/passphrase/credentialtype/PassphraseSSHPrivateKeyTextCredentialType.php
@@ -47,7 +47,7 @@ final class PassphraseSSHPrivateKeyTextCredentialType
         pht(
           'Decrypting SSH keys requires the `%s` binary, but it '.
           'is not available in %s. Either make it available or strip the '.
-          'password fromt his SSH key manually before uploading it.',
+          'password from this SSH key manually before uploading it.',
           'ssh-keygen',
           '$PATH'));
     }
diff --git a/src/applications/passphrase/view/PassphraseCredentialControl.php b/src/applications/passphrase/view/PassphraseCredentialControl.php
index 9c9706fe3f..8ba1ef9bc8 100644
--- a/src/applications/passphrase/view/PassphraseCredentialControl.php
+++ b/src/applications/passphrase/view/PassphraseCredentialControl.php
@@ -67,7 +67,7 @@ final class PassphraseCredentialControl extends AphrontFormControl {
             $user_credential->getName());
         }
       } catch (PhabricatorPolicyException $policy_exception) {
-        // Pull the credential with the ominipotent viewer so we can look up
+        // Pull the credential with the omnipotent viewer so we can look up
         // the ID and provide the monogram.
         $omnipotent_credential = id(new PassphraseCredentialQuery())
           ->setViewer(PhabricatorUser::getOmnipotentUser())
diff --git a/src/applications/people/markup/PhabricatorMentionRemarkupRule.php b/src/applications/people/markup/PhabricatorMentionRemarkupRule.php
index aa5b7f908e..60d4a8168f 100644
--- a/src/applications/people/markup/PhabricatorMentionRemarkupRule.php
+++ b/src/applications/people/markup/PhabricatorMentionRemarkupRule.php
@@ -10,7 +10,7 @@ final class PhabricatorMentionRemarkupRule extends PhutilRemarkupRule {
 
   // NOTE: The negative lookbehind prevents matches like "mail@lists", while
   // allowing constructs like "@tomo/@mroch". Since we now allow periods in
-  // usernames, we can't resonably distinguish that "@company.com" isn't a
+  // usernames, we can't reasonably distinguish that "@company.com" isn't a
   // username, so we'll incorrectly pick it up, but there's little to be done
   // about that. We forbid terminal periods so that we can correctly capture
   // "@joe" instead of "@joe." in "Hey, @joe.".
diff --git a/src/applications/people/query/PhabricatorPeopleQuery.php b/src/applications/people/query/PhabricatorPeopleQuery.php
index e756a9a4fd..35ff480833 100644
--- a/src/applications/people/query/PhabricatorPeopleQuery.php
+++ b/src/applications/people/query/PhabricatorPeopleQuery.php
@@ -425,7 +425,7 @@ final class PhabricatorPeopleQuery
         }
 
         // If the user is set to "Available" for this event, don't consider it
-        // when computin their away status.
+        // when computing their away status.
         if (!$invitee->getDisplayAvailability($event)) {
           continue;
         }
@@ -492,7 +492,7 @@ final class PhabricatorPeopleQuery
         // valid for that long.
 
         // NOTE: This doesn't handle overlapping events with the greatest
-        // possible care. In theory, if you're attenting multiple events
+        // possible care. In theory, if you're attending multiple events
         // simultaneously we should accommodate that. However, it's complex
         // to compute, rare, and probably not confusing most of the time.
 
diff --git a/src/applications/people/query/PhabricatorPeopleSearchEngine.php b/src/applications/people/query/PhabricatorPeopleSearchEngine.php
index fa363c4428..0a4367d367 100644
--- a/src/applications/people/query/PhabricatorPeopleSearchEngine.php
+++ b/src/applications/people/query/PhabricatorPeopleSearchEngine.php
@@ -79,7 +79,7 @@ final class PhabricatorPeopleSearchEngine
         ->setOptions(
           pht('(Show All)'),
           pht('Show Only Unapproved Users'),
-          pht('Hide Unappproved Users'))
+          pht('Hide Unapproved Users'))
         ->setDescription(
           pht(
             'Pass true to find only users awaiting administrative approval, '.
diff --git a/src/applications/people/storage/PhabricatorUser.php b/src/applications/people/storage/PhabricatorUser.php
index 166feaa237..1745154826 100644
--- a/src/applications/people/storage/PhabricatorUser.php
+++ b/src/applications/people/storage/PhabricatorUser.php
@@ -407,7 +407,7 @@ final class PhabricatorUser
     $token = substr($token, $breach_prelen + self::CSRF_SALT_LENGTH);
 
     // When the user posts a form, we check that it contains a valid CSRF token.
-    // Tokens cycle each hour (every CSRF_CYLCE_FREQUENCY seconds) and we accept
+    // Tokens cycle each hour (every CSRF_CYCLE_FREQUENCY seconds) and we accept
     // either the current token, the next token (users can submit a "future"
     // token if you have two web frontends that have some clock skew) or any of
     // the last 6 tokens. This means that pages are valid for up to 7 hours.
@@ -1180,7 +1180,7 @@ final class PhabricatorUser
   /**
    * Get a scalar string identifying this user.
    *
-   * This is similar to using the PHID, but distinguishes between ominpotent
+   * This is similar to using the PHID, but distinguishes between omnipotent
    * and public users explicitly. This allows safe construction of cache keys
    * or cache buckets which do not conflate public and omnipotent users.
    *
@@ -1459,7 +1459,7 @@ final class PhabricatorUser
       id(new PhabricatorConduitSearchFieldSpecification())
         ->setKey('roles')
         ->setType('list<string>')
-        ->setDescription(pht('List of acccount roles.')),
+        ->setDescription(pht('List of account roles.')),
     );
   }
 
diff --git a/src/applications/phid/handle/pool/PhabricatorHandleList.php b/src/applications/phid/handle/pool/PhabricatorHandleList.php
index d9a279f8ba..a7b404c5a7 100644
--- a/src/applications/phid/handle/pool/PhabricatorHandleList.php
+++ b/src/applications/phid/handle/pool/PhabricatorHandleList.php
@@ -82,7 +82,7 @@ final class PhabricatorHandleList
       if (!isset($this[$phid])) {
         throw new Exception(
           pht(
-            'Trying to create a new sublist of an existsing handle list, '.
+            'Trying to create a new sublist of an existing handle list, '.
             'but PHID "%s" does not appear in the parent list.',
             $phid));
       }
diff --git a/src/applications/phid/resolver/PhabricatorPHIDResolver.php b/src/applications/phid/resolver/PhabricatorPHIDResolver.php
index 9070139a0b..940221f551 100644
--- a/src/applications/phid/resolver/PhabricatorPHIDResolver.php
+++ b/src/applications/phid/resolver/PhabricatorPHIDResolver.php
@@ -3,7 +3,7 @@
 /**
  * Resolve a list of identifiers into PHIDs.
  *
- * This class simplifies the process of convering a list of mixed token types
+ * This class simplifies the process of converting a list of mixed token types
  * (like some PHIDs and some usernames) into a list of just PHIDs.
  */
 abstract class PhabricatorPHIDResolver extends Phobject {
diff --git a/src/applications/phortune/provider/PhortuneStripePaymentProvider.php b/src/applications/phortune/provider/PhortuneStripePaymentProvider.php
index 1e7c348297..5adfc62e19 100644
--- a/src/applications/phortune/provider/PhortuneStripePaymentProvider.php
+++ b/src/applications/phortune/provider/PhortuneStripePaymentProvider.php
@@ -361,7 +361,7 @@ final class PhortuneStripePaymentProvider extends PhortunePaymentProvider {
       case 'error:duplicate_transaction':
       case 'error:processing_error':
       default:
-        // NOTE: These errors currently don't recevive a detailed message.
+        // NOTE: These errors currently don't receive a detailed message.
         // NOTE: We can also end up here with "http:nnn" messages.
 
         // TODO: At least some of these should have a better message, or be
diff --git a/src/applications/phrequent/storage/PhrequentTimeBlock.php b/src/applications/phrequent/storage/PhrequentTimeBlock.php
index ee5fd00a44..3800651476 100644
--- a/src/applications/phrequent/storage/PhrequentTimeBlock.php
+++ b/src/applications/phrequent/storage/PhrequentTimeBlock.php
@@ -124,7 +124,7 @@ final class PhrequentTimeBlock extends Phobject {
               }
             } else {
               // Here, we've prematurely ended a deeper stratum. Merge higher
-              // stata. This looks like this:
+              // strata. This looks like this:
               //
               //                V
               //                V
diff --git a/src/applications/phriction/editor/PhrictionTransactionEditor.php b/src/applications/phriction/editor/PhrictionTransactionEditor.php
index 1920b4c718..fcc9fe0474 100644
--- a/src/applications/phriction/editor/PhrictionTransactionEditor.php
+++ b/src/applications/phriction/editor/PhrictionTransactionEditor.php
@@ -452,7 +452,7 @@ final class PhrictionTransactionEditor
     $verb) {
 
     $errors = array();
-    // NOTE: We use the ominpotent user for these checks because policy
+    // NOTE: We use the omnipotent user for these checks because policy
     // doesn't matter; existence does.
     $other_doc_viewer = PhabricatorUser::getOmnipotentUser();
     $ancestral_slugs = PhabricatorSlug::getAncestry($object->getSlug());
diff --git a/src/applications/policy/filter/PhabricatorPolicyFilter.php b/src/applications/policy/filter/PhabricatorPolicyFilter.php
index f3e1bd2083..fb03936ec2 100644
--- a/src/applications/policy/filter/PhabricatorPolicyFilter.php
+++ b/src/applications/policy/filter/PhabricatorPolicyFilter.php
@@ -257,7 +257,7 @@ final class PhabricatorPolicyFilter extends Phobject {
       $filtered[$key] = $object;
     }
 
-    // If we survied the primary checks, apply extended checks to objects
+    // If we survived the primary checks, apply extended checks to objects
     // with extended policies.
     $results = array();
     $extended = array();
diff --git a/src/applications/project/query/PhabricatorProjectColumnQuery.php b/src/applications/project/query/PhabricatorProjectColumnQuery.php
index 41ea770137..13f2f52a43 100644
--- a/src/applications/project/query/PhabricatorProjectColumnQuery.php
+++ b/src/applications/project/query/PhabricatorProjectColumnQuery.php
@@ -97,7 +97,7 @@ final class PhabricatorProjectColumnQuery
       if ($proxy_phid !== null) {
         $proxy = idx($proxies, $proxy_phid);
 
-        // Only attach valid proxies, so we don't end up getting surprsied if
+        // Only attach valid proxies, so we don't end up getting surprised if
         // an install somehow gets junk into their database.
         if (!($proxy instanceof PhabricatorColumnProxyInterface)) {
           $proxy = null;
diff --git a/src/applications/project/remarkup/ProjectRemarkupRule.php b/src/applications/project/remarkup/ProjectRemarkupRule.php
index 70d6b8eda3..96092bab86 100644
--- a/src/applications/project/remarkup/ProjectRemarkupRule.php
+++ b/src/applications/project/remarkup/ProjectRemarkupRule.php
@@ -24,7 +24,7 @@ final class ProjectRemarkupRule extends PhabricatorObjectRemarkupRule {
   protected function getObjectIDPattern() {
     // NOTE: The latter half of this rule matches monograms with internal
     // periods, like `#domain.com`, but does not match monograms with terminal
-    // periods, because they're probably just puncutation.
+    // periods, because they're probably just punctuation.
 
     // Broadly, this will not match every possible project monogram, and we
     // accept some false negatives -- like `#dot.` -- in order to avoid a bunch
diff --git a/src/applications/releeph/differential/DifferentialReleephRequestFieldSpecification.php b/src/applications/releeph/differential/DifferentialReleephRequestFieldSpecification.php
index dd67f5bc59..56d371f1d6 100644
--- a/src/applications/releeph/differential/DifferentialReleephRequestFieldSpecification.php
+++ b/src/applications/releeph/differential/DifferentialReleephRequestFieldSpecification.php
@@ -5,7 +5,7 @@
  *
  * 1: To parse "Releeph: picks RQ<nn>" headers in commits created by
  * arc-releeph so that RQs committed by arc-releeph have real
- * PhabricatorRepositoryCommits associated with them (instaed of just the SHA
+ * PhabricatorRepositoryCommits associated with them (instead of just the SHA
  * of the commit, as seen by the pusher).
  *
  * 2: If requestors want to commit directly to their release branch, they can
diff --git a/src/applications/releeph/field/selector/ReleephDefaultFieldSelector.php b/src/applications/releeph/field/selector/ReleephDefaultFieldSelector.php
index df5e075ebc..af18bfd3d6 100644
--- a/src/applications/releeph/field/selector/ReleephDefaultFieldSelector.php
+++ b/src/applications/releeph/field/selector/ReleephDefaultFieldSelector.php
@@ -7,7 +7,7 @@ final class ReleephDefaultFieldSelector extends ReleephFieldSelector {
    *
    * TODO: This is a giant hacky mess because I am dumb and moved forward on
    * Releeph changes with partial information. Recover from this as gracefully
-   * as possible. This obivously is an abomination. -epriestley
+   * as possible. This obviously is an abomination. -epriestley
    */
   public static function isFacebook() {
     return class_exists('ReleephFacebookKarmaFieldSpecification');
diff --git a/src/applications/repository/daemon/PhabricatorMercurialGraphStream.php b/src/applications/repository/daemon/PhabricatorMercurialGraphStream.php
index 33f947e2f8..205b267ed5 100644
--- a/src/applications/repository/daemon/PhabricatorMercurialGraphStream.php
+++ b/src/applications/repository/daemon/PhabricatorMercurialGraphStream.php
@@ -113,7 +113,7 @@ final class PhabricatorMercurialGraphStream
   private function parseParents($parents, $target_rev) {
 
     // The hg '{parents}' token is empty if there is one "natural" parent
-    // (predecessor local commit ID). Othwerwise, it may have one or two
+    // (predecessor local commit ID). Otherwise, it may have one or two
     // parents. The string looks like this:
     //
     //  151:1f6c61a60586 154:1d5f799ebe1e
diff --git a/src/applications/repository/data/PhabricatorRepositoryURINormalizer.php b/src/applications/repository/data/PhabricatorRepositoryURINormalizer.php
index 27e93535ab..a29d110a67 100644
--- a/src/applications/repository/data/PhabricatorRepositoryURINormalizer.php
+++ b/src/applications/repository/data/PhabricatorRepositoryURINormalizer.php
@@ -27,10 +27,10 @@
  *     // URIs are very unlikely to be the same repository.
  *   }
  *
- * Because a repository can be hosted at arbitrarly many arbitrary URIs, there
+ * Because a repository can be hosted at arbitrarily many arbitrary URIs, there
  * is no way to completely prevent false negatives by only examining URIs
  * (that is, repositories with totally different URIs could really be the same).
- * However, normalization is relatively agressive and false negatives should
+ * However, normalization is relatively aggressive and false negatives should
  * be rare: if normalization says two URIs are different repositories, they
  * probably are.
  *
diff --git a/src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php b/src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php
index d0b84a6b31..8e99aabafa 100644
--- a/src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php
+++ b/src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php
@@ -703,7 +703,7 @@ final class PhabricatorRepositoryDiscoveryEngine
     // new repository for the first time is less important than any other
     // daemon task.
 
-    // If the repostitory has finished importing and we're just catching up
+    // If the repository has finished importing and we're just catching up
     // on recent commits, we schedule discovery at COMMIT priority, which is
     // slightly below the default priority.
 
diff --git a/src/applications/repository/management/PhabricatorRepositoryManagementThawWorkflow.php b/src/applications/repository/management/PhabricatorRepositoryManagementThawWorkflow.php
index 2f09ffd9be..71076314da 100644
--- a/src/applications/repository/management/PhabricatorRepositoryManagementThawWorkflow.php
+++ b/src/applications/repository/management/PhabricatorRepositoryManagementThawWorkflow.php
@@ -90,7 +90,7 @@ final class PhabricatorRepositoryManagementThawWorkflow
       $risk_message);
 
     $is_force = $args->getArg('force');
-    $prompt = pht('Accept the possibilty of permanent data loss?');
+    $prompt = pht('Accept the possibility of permanent data loss?');
     if (!$is_force && !phutil_console_confirm($prompt)) {
       throw new PhutilArgumentUsageException(
         pht('User aborted the workflow.'));
diff --git a/src/applications/repository/storage/PhabricatorRepository.php b/src/applications/repository/storage/PhabricatorRepository.php
index 09fb239bf0..713ed92270 100644
--- a/src/applications/repository/storage/PhabricatorRepository.php
+++ b/src/applications/repository/storage/PhabricatorRepository.php
@@ -1875,7 +1875,7 @@ final class PhabricatorRepository extends PhabricatorRepositoryDAO
 
 
   /**
-   * Retrieve the sevice URI for the device hosting this repository.
+   * Retrieve the service URI for the device hosting this repository.
    *
    * See @{method:newConduitClient} for a general discussion of interacting
    * with repository services. This method provides lower-level resolution of
@@ -2129,7 +2129,7 @@ final class PhabricatorRepository extends PhabricatorRepositoryDAO
     if ($this->isGit()) {
       // $_ENV does not populate in CLI contexts if "E" is missing from
       // "variables_order" in PHP config. Currently, we do not require this
-      // to be configured. Since it may not be, explictitly bring expected Git
+      // to be configured. Since it may not be, explicitly bring expected Git
       // environmental variables into scope. This list is not exhaustive, but
       // only lists variables with a known impact on commit hook behavior.
 
diff --git a/src/applications/repository/storage/PhabricatorRepositoryURI.php b/src/applications/repository/storage/PhabricatorRepositoryURI.php
index cbd25b74eb..c8d560705a 100644
--- a/src/applications/repository/storage/PhabricatorRepositoryURI.php
+++ b/src/applications/repository/storage/PhabricatorRepositoryURI.php
@@ -362,7 +362,7 @@ final class PhabricatorRepositoryURI
       return PhabricatorEnv::getEnvConfig('diffusion.ssh-port');
     }
 
-    // If Phabricator is running on a nonstandard port, use that as the defualt
+    // If Phabricator is running on a nonstandard port, use that as the default
     // port for URIs with the same protocol.
 
     $is_http = ($protocol == self::BUILTIN_PROTOCOL_HTTP);
diff --git a/src/applications/repository/storage/PhabricatorRepositoryWorkingCopyVersion.php b/src/applications/repository/storage/PhabricatorRepositoryWorkingCopyVersion.php
index 51abc70d35..750cc91c47 100644
--- a/src/applications/repository/storage/PhabricatorRepositoryWorkingCopyVersion.php
+++ b/src/applications/repository/storage/PhabricatorRepositoryWorkingCopyVersion.php
@@ -66,7 +66,7 @@ final class PhabricatorRepositoryWorkingCopyVersion
    * may have committed and acknowledged a write on a node that lost the lock
    * partway through the write and is no longer reachable.
    *
-   * In particular, if a node loses its connection to the datbase the global
+   * In particular, if a node loses its connection to the database the global
    * lock is released by default. This is a durable lock which stays locked
    * by default.
    */
diff --git a/src/applications/repository/worker/commitchangeparser/PhabricatorRepositoryMercurialCommitChangeParserWorker.php b/src/applications/repository/worker/commitchangeparser/PhabricatorRepositoryMercurialCommitChangeParserWorker.php
index 0fdd18b20f..60e6c5a5e6 100644
--- a/src/applications/repository/worker/commitchangeparser/PhabricatorRepositoryMercurialCommitChangeParserWorker.php
+++ b/src/applications/repository/worker/commitchangeparser/PhabricatorRepositoryMercurialCommitChangeParserWorker.php
@@ -216,7 +216,7 @@ final class PhabricatorRepositoryMercurialCommitChangeParserWorker
           $changes[$path]['changeType']
             = DifferentialChangeType::TYPE_COPY_AWAY;
         } else if ($existing_type == DifferentialChangeType::TYPE_ADD) {
-          // This change removes a diretory and replaces it with a file. Mark
+          // This change removes a directory and replaces it with a file. Mark
           // it as "change" instead of "add".
           $changes[$path]['changeType'] = DifferentialChangeType::TYPE_CHANGE;
         }
diff --git a/src/applications/search/engine/PhabricatorApplicationSearchEngine.php b/src/applications/search/engine/PhabricatorApplicationSearchEngine.php
index f4c460ce34..c0497a0c03 100644
--- a/src/applications/search/engine/PhabricatorApplicationSearchEngine.php
+++ b/src/applications/search/engine/PhabricatorApplicationSearchEngine.php
@@ -359,7 +359,7 @@ abstract class PhabricatorApplicationSearchEngine extends Phobject {
    *   );
    *
    * Any unspecified fields (including custom fields and fields added
-   * automatically by infrastruture) will be put in the middle.
+   * automatically by infrastructure) will be put in the middle.
    *
    * @return list<string> Default ordering for field keys.
    */
diff --git a/src/applications/search/management/PhabricatorSearchManagementWorkflow.php b/src/applications/search/management/PhabricatorSearchManagementWorkflow.php
index 43dad986ea..b09f4b2105 100644
--- a/src/applications/search/management/PhabricatorSearchManagementWorkflow.php
+++ b/src/applications/search/management/PhabricatorSearchManagementWorkflow.php
@@ -5,7 +5,7 @@ abstract class PhabricatorSearchManagementWorkflow
 
   protected function validateClusterSearchConfig() {
     // Configuration is normally validated by setup self-checks on the web
-    // workflow, but users may reasonsably run `bin/search` commands after
+    // workflow, but users may reasonably run `bin/search` commands after
     // making manual edits to "local.json". Re-verify configuration here before
     // continuing.
 
diff --git a/src/applications/search/menuitem/PhabricatorMotivatorProfileMenuItem.php b/src/applications/search/menuitem/PhabricatorMotivatorProfileMenuItem.php
index e67d22b555..071979d391 100644
--- a/src/applications/search/menuitem/PhabricatorMotivatorProfileMenuItem.php
+++ b/src/applications/search/menuitem/PhabricatorMotivatorProfileMenuItem.php
@@ -80,7 +80,7 @@ final class PhabricatorMotivatorProfileMenuItem
   private function getCatFacts() {
     return array(
       pht('Cats purr when they are happy, upset, or asleep.'),
-      pht('The first cats evolved on the savanah about 8,000 years ago.'),
+      pht('The first cats evolved on the savannah about 8,000 years ago.'),
       pht(
         'Cats have a tail, two feet, between one and three ears, and two '.
         'other feet.'),
@@ -111,7 +111,7 @@ final class PhabricatorMotivatorProfileMenuItem
       pht(
         'Not all cats can retract their claws, but most of them can.'),
       pht(
-        'In the wild, cats and racoons sometimes hunt together in packs.'),
+        'In the wild, cats and raccoons sometimes hunt together in packs.'),
       pht(
         'The Spanish word for cat is "cato". The biggest cat is called '.
         '"el cato".'),
diff --git a/src/applications/settings/panel/PhabricatorSettingsPanel.php b/src/applications/settings/panel/PhabricatorSettingsPanel.php
index d2008f8857..19ac6fec62 100644
--- a/src/applications/settings/panel/PhabricatorSettingsPanel.php
+++ b/src/applications/settings/panel/PhabricatorSettingsPanel.php
@@ -143,7 +143,7 @@ abstract class PhabricatorSettingsPanel extends Phobject {
   /**
    * Return false to prevent this panel from being displayed or used. You can
    * do, e.g., configuration checks here, to determine if the feature your
-   * panel controls is unavailble in this install. By default, all panels are
+   * panel controls is unavailable in this install. By default, all panels are
    * enabled.
    *
    * @return bool True if the panel should be shown.
diff --git a/src/applications/settings/query/PhabricatorUserPreferencesQuery.php b/src/applications/settings/query/PhabricatorUserPreferencesQuery.php
index de4887cbb8..1a6133724d 100644
--- a/src/applications/settings/query/PhabricatorUserPreferencesQuery.php
+++ b/src/applications/settings/query/PhabricatorUserPreferencesQuery.php
@@ -49,7 +49,7 @@ final class PhabricatorUserPreferencesQuery
    * If no settings exist for a user, a new empty settings object with
    * appropriate defaults is returned.
    *
-   * @param bool True to generat synthetic preferences for missing users.
+   * @param bool True to generate synthetic preferences for missing users.
    */
   public function needSyntheticPreferences($synthetic) {
     $this->synthetic = $synthetic;
diff --git a/src/applications/settings/setting/PhabricatorTimezoneSetting.php b/src/applications/settings/setting/PhabricatorTimezoneSetting.php
index b7a6b94e30..887e08129b 100644
--- a/src/applications/settings/setting/PhabricatorTimezoneSetting.php
+++ b/src/applications/settings/setting/PhabricatorTimezoneSetting.php
@@ -46,7 +46,7 @@ final class PhabricatorTimezoneSetting
 
     throw new Exception(
       pht(
-        'Timezone "%s" is not a valid timezone identiifer.',
+        'Timezone "%s" is not a valid timezone identifier.',
         $value));
   }
 
diff --git a/src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php b/src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php
index e005c120f0..817e92ce39 100644
--- a/src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php
+++ b/src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php
@@ -83,7 +83,7 @@ final class PhabricatorSubscriptionsEditController
     } else {
 
       // TODO: Eventually, get rid of this once everything implements
-      // PhabriatorApplicationTransactionInterface.
+      // PhabricatorApplicationTransactionInterface.
 
       $editor = id(new PhabricatorSubscriptionsEditor())
         ->setActor($viewer)
diff --git a/src/applications/transactions/editengine/PhabricatorEditEngine.php b/src/applications/transactions/editengine/PhabricatorEditEngine.php
index e1d6812778..6c9cb5e201 100644
--- a/src/applications/transactions/editengine/PhabricatorEditEngine.php
+++ b/src/applications/transactions/editengine/PhabricatorEditEngine.php
@@ -763,7 +763,7 @@ abstract class PhabricatorEditEngine
    * Load an object given a configured query.
    *
    * @param PhabricatorPolicyAwareQuery Configured query.
-   * @param list<const> List of required capabilitiy constants, or omit for
+   * @param list<const> List of required capability constants, or omit for
    *  defaults.
    * @return object|null Object, or null if no such object exists.
    * @task load
@@ -1382,7 +1382,7 @@ abstract class PhabricatorEditEngine
    * and that field is visible and editable for the user.
    *
    * For example, you can use it to test if a user is able to reassign tasks
-   * or not, prior to rendering dedicated UI for task reassingment.
+   * or not, prior to rendering dedicated UI for task reassignment.
    *
    * Note that this method does NOT test if the user can actually edit the
    * current object, just if they have access to the related field.
diff --git a/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php b/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
index 07edf5b0c8..175a5775ee 100644
--- a/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
+++ b/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
@@ -3701,7 +3701,7 @@ abstract class PhabricatorApplicationTransactionEditor
 
       // If a later project in the list is an ancestor of this one, it will
       // have added itself to the map. If any ancestor of this project points
-      // at itself in the map, this project should be dicarded in favor of
+      // at itself in the map, this project should be discarded in favor of
       // that later ancestor.
       foreach ($project->getAncestorProjects() as $ancestor) {
         $ancestor_phid = $ancestor->getPHID();
diff --git a/src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php b/src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php
index 6956561490..8b747281b3 100644
--- a/src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php
+++ b/src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php
@@ -451,7 +451,7 @@ abstract class PhabricatorTypeaheadDatasource extends Phobject {
   public static function isFunctionToken($token) {
     // We're looking for a "(" so that a string like "members(q" is identified
     // and parsed as a function call. This allows us to start generating
-    // results immeidately, before the user fully types out "members(quack)".
+    // results immediately, before the user fully types out "members(quack)".
     return (strpos($token, '(') !== false);
   }
 
diff --git a/src/applications/uiexample/examples/PhabricatorStatusUIExample.php b/src/applications/uiexample/examples/PhabricatorStatusUIExample.php
index 6583bc5848..76a345b8e3 100644
--- a/src/applications/uiexample/examples/PhabricatorStatusUIExample.php
+++ b/src/applications/uiexample/examples/PhabricatorStatusUIExample.php
@@ -33,7 +33,7 @@ final class PhabricatorStatusUIExample extends PhabricatorUIExample {
     $view->addItem(
       id(new PHUIStatusItemView())
         ->setIcon(PHUIStatusItemView::ICON_WARNING, 'dark', pht('Caution'))
-        ->setTarget(pht('Pomegranite'))
+        ->setTarget(pht('Pomegranate'))
         ->setNote(pht('Lots of seeds. Watch out.')));
 
     $view->addItem(
diff --git a/src/docs/contributor/cla.diviner b/src/docs/contributor/cla.diviner
index f08711483a..54e0142beb 100644
--- a/src/docs/contributor/cla.diviner
+++ b/src/docs/contributor/cla.diviner
@@ -87,7 +87,7 @@ you to consent. If you are not comfortable with this, do not sign the CLA and
 do not contribute to Phabricator.
 
 **Limitation of Liability**: The second benefit the CLA provides is that it
-makes the terms of your contribition explicitly clear upfront, and it puts us
+makes the terms of your contribution explicitly clear upfront, and it puts us
 in a much stronger legal position if a contributor later claims there is
 ambiguity about ownership of their work. We can point at the document they
 signed as proof that they consented to our use and understood the terms of
@@ -100,7 +100,7 @@ anything we are likely to face, but SCO claimed billions of dollars in damages
 and the litigation has now been ongoing for more than a decade.
 
 We want to avoid situations like this in the future by making the terms of
-contibution explicit upfront.
+contribution explicit upfront.
 
 Generally, we believe the terms of the CLA are fair and reasonable for
 contributors, and that the primary way contributors benefit from contributing
@@ -158,7 +158,7 @@ local law.
 
 If you are unsure, you should speak with your employer or a lawyer. If you
 contribute code you do not own under the individual CLA, you are exposing
-yourself to liability. You may also be exposing us to liablity, but we'll have
+yourself to liability. You may also be exposing us to liability, but we'll have
 the CLA on our side to show that we were unwilling pawns in your malicious
 scheme to defraud your employer.
 
diff --git a/src/docs/contributor/describing_problems.diviner b/src/docs/contributor/describing_problems.diviner
index 57bb849b02..d06d7b7d64 100644
--- a/src/docs/contributor/describing_problems.diviner
+++ b/src/docs/contributor/describing_problems.diviner
@@ -140,7 +140,7 @@ Additional Resources
 
 Poor problem descriptions are a common issue in software development and
 extensively documented elsewhere. Here are some additional resources describing
-how to describe problems and ask questions effectivey:
+how to describe problems and ask questions effectively:
 
   - [[ http://www.catb.org/esr/faqs/smart-questions.html | How To Ask
     Questions The Smart Way ]], by Eric S. Raymond
diff --git a/src/docs/contributor/internationalization.diviner b/src/docs/contributor/internationalization.diviner
index e9cdb9dfcd..9c78eb60fb 100644
--- a/src/docs/contributor/internationalization.diviner
+++ b/src/docs/contributor/internationalization.diviner
@@ -198,7 +198,7 @@ would technically be English and most English readers would understand the
 meaning, but no native English speaker would speak or write like this.
 
 However, some languages have different subject-verb order rules or
-colloquisalisms, and a word order which transliterates like this may sound more
+colloquialisms, and a word order which transliterates like this may sound more
 natural to a native speaker. By translating fragments instead of complete
 sentences, you lock translators into English word order.
 
diff --git a/src/docs/contributor/reproduction_steps.diviner b/src/docs/contributor/reproduction_steps.diviner
index d5d58913b7..1050e43c48 100644
--- a/src/docs/contributor/reproduction_steps.diviner
+++ b/src/docs/contributor/reproduction_steps.diviner
@@ -112,7 +112,7 @@ we'll probably be able to follow them and reproduce the issue ourselves.
 
 If you can't follow your steps because they depend on information which is not
 available on a clean instance (for example, a certain config setting), rewrite
-them to include instrutions to create that information (for example, adjusting
+them to include instructions to create that information (for example, adjusting
 the config to the problematic value).
 
 If you follow your instructions but the issue does not reproduce, the issue
@@ -191,7 +191,7 @@ smaller case which reproduces the problem, which might be something like this:
   - Files like `A`, which have uppercase letters, do not work.
 
 With a simpler reproduction scenario, you can simplify your steps to be more
-tailored and mimimal. This will help us pointpoint the issue more quickly and
+tailored and minimal. This will help us pinpoint the issue more quickly and
 be more certain that we've understood and resolved it.
 
 It is more important that steps be complete than that they be simple, and it's
diff --git a/src/docs/user/cluster/cluster.diviner b/src/docs/user/cluster/cluster.diviner
index 30ad53efb8..2362b80485 100644
--- a/src/docs/user/cluster/cluster.diviner
+++ b/src/docs/user/cluster/cluster.diviner
@@ -225,7 +225,7 @@ Cluster: Notifications
 Configuring multiple notification hosts is simple and has no pre-requisites.
 
 With multiple notification hosts, you can survive the loss of any subset of
-hosts as long as at least one host remains alive. Service may be breifly
+hosts as long as at least one host remains alive. Service may be briefly
 disrupted directly after the incident which destroys the other hosts.
 
 Notifications are noncritical, so this normally has little practical impact
diff --git a/src/docs/user/cluster/cluster_databases.diviner b/src/docs/user/cluster/cluster_databases.diviner
index 7255662958..51bf14d925 100644
--- a/src/docs/user/cluster/cluster_databases.diviner
+++ b/src/docs/user/cluster/cluster_databases.diviner
@@ -31,7 +31,7 @@ although this may change in the future.
 
 Phabricator applications //can// be partitioned across multiple database
 masters. This does not provide redundancy and generally does not increase
-resiliance or resistance to data loss, but can help you scale and operate
+resilience or resistance to data loss, but can help you scale and operate
 Phabricator. For details, see
 @{article:Cluster: Partitioning and Advanced Configuration}.
 
diff --git a/src/docs/user/cluster/cluster_devices.diviner b/src/docs/user/cluster/cluster_devices.diviner
index 57729fdb86..c90aa220c4 100644
--- a/src/docs/user/cluster/cluster_devices.diviner
+++ b/src/docs/user/cluster/cluster_devices.diviner
@@ -212,7 +212,7 @@ $ ./bin/almanac register \
 For example, you might run this command on `repo001` when using a shared key:
 
 ```
-$ ./bin/almanac register
+$ ./bin/almanac register \
   --device keywarden.mycompany.net \
   --private-key /path/to/private-key \
   --identify-as repo001.mycompany.net
diff --git a/src/docs/user/cluster/cluster_partitioning.diviner b/src/docs/user/cluster/cluster_partitioning.diviner
index 4dbb47f9d4..20ae11d6a6 100644
--- a/src/docs/user/cluster/cluster_partitioning.diviner
+++ b/src/docs/user/cluster/cluster_partitioning.diviner
@@ -21,7 +21,7 @@ This configuration is complex, and very few installs will benefit from pursuing
 it. Phabricator will normally run comfortably with a single database master
 even for large organizations.
 
-Partitioning generally does not do much to increase resiliance or make it
+Partitioning generally does not do much to increase resilience or make it
 easier to recover from disasters, and is primarily a mechanism for scaling and
 operational convenience.
 
@@ -212,14 +212,14 @@ only one master.
 
 `persistent` //(bool)// Enables persistent connections. Defaults to off.
 
-With persitent connections enabled, Phabricator will keep a pool of database
+With persistent connections enabled, Phabricator will keep a pool of database
 connections open between web requests and reuse them when serving subsequent
 requests.
 
 The primary benefit of using persistent connections is that it will greatly
 reduce pressure on how quickly outbound TCP ports are opened and closed. After
 a TCP port closes, it normally can't be used again for about 60 seconds, so
-rapidly cycling ports can cause resource exuastion. If you're seeing failures
+rapidly cycling ports can cause resource exhaustion. If you're seeing failures
 because requests are unable to bind to an outbound port, enabling this option
 is likely to fix the issue. This option may also slightly increase performance.
 
diff --git a/src/docs/user/cluster/cluster_repositories.diviner b/src/docs/user/cluster/cluster_repositories.diviner
index 21aba731cc..44003679af 100644
--- a/src/docs/user/cluster/cluster_repositories.diviner
+++ b/src/docs/user/cluster/cluster_repositories.diviner
@@ -440,7 +440,7 @@ device or devices have up-to-date information.
 
 When Phabricator can not tell which device in a cluster is a leader, it freezes
 the cluster because it is possible that some devices have less data and others
-have more, and if it choses a leader arbitrarily it may destroy some data
+have more, and if it chooses a leader arbitrarily it may destroy some data
 which you would prefer to retain.
 
 To resolve this, you need to tell Phabricator which device has the most
diff --git a/src/docs/user/cluster/cluster_search.diviner b/src/docs/user/cluster/cluster_search.diviner
index c658f50db4..25c35aa34a 100644
--- a/src/docs/user/cluster/cluster_search.diviner
+++ b/src/docs/user/cluster/cluster_search.diviner
@@ -100,14 +100,14 @@ A typical `mysql` service configuration looks like this:
 Service Type: Elasticsearch
 ======================
 
-The `elasticsearch` sevice type supports these options:
+The `elasticsearch` service type supports these options:
 
 | Key | Description |
 |---|---|
 | `protocol` | Either `"http"` (default) or `"https"`.
 | `port` | Elasticsearch TCP port.
 | `version` | Elasticsearch version, either `2` or `5` (default).
-| `path` | Path for the index. Defaults to `/phabriator`. Advanced.
+| `path` | Path for the index. Defaults to `/phabricator`. Advanced.
 
 A typical `elasticsearch` service configuration looks like this:
 
diff --git a/src/docs/user/configuration/configuration_guide.diviner b/src/docs/user/configuration/configuration_guide.diviner
index 98c27b9736..8b221bda41 100644
--- a/src/docs/user/configuration/configuration_guide.diviner
+++ b/src/docs/user/configuration/configuration_guide.diviner
@@ -116,7 +116,7 @@ Restart nginx after making your edits, then continue to "Setup" below.
 
 = Webserver: Configuring lighttpd =
 
-NOTE: Follow these instructions to use lighttpd. To use Apache or niginx, scroll
+NOTE: Follow these instructions to use lighttpd. To use Apache or nginx, scroll
 up to their sections.
 
 For lighttpd, add a section like this to your lighttpd.conf:
@@ -138,7 +138,7 @@ server.modules list:
 
 Finally, you should run the following commands to enable php support:
 
-  $ sudo apt-get install php5-cgi  # for ubuntu; other distros should be similar
+  $ sudo apt-get install php5-cgi  # for Ubuntu; other distros should be similar
   $ sudo lighty-enable-mod fastcgi-php
 
 Restart lighttpd after making your edits, then continue below.
diff --git a/src/docs/user/configuration/configuring_backups.diviner b/src/docs/user/configuration/configuring_backups.diviner
index d32eebb0da..9c283c5722 100644
--- a/src/docs/user/configuration/configuring_backups.diviner
+++ b/src/docs/user/configuration/configuring_backups.diviner
@@ -64,7 +64,7 @@ If you host repositories in Phabricator, you should back them up. You can use
 repository. To back them up, copy them elsewhere.
 
 You can also just clone them and keep the clones up to date, or use
-{nav Add Mirror} to have the mirror somewhere automatically.
+{nav Add Mirror} to have them mirror somewhere automatically.
 
 
 Restore: Hosted Repositories
diff --git a/src/docs/user/configuration/configuring_encryption.diviner b/src/docs/user/configuration/configuring_encryption.diviner
index dbd0e76314..36315506e6 100644
--- a/src/docs/user/configuration/configuring_encryption.diviner
+++ b/src/docs/user/configuration/configuring_encryption.diviner
@@ -80,7 +80,7 @@ Format: Raw Data
 ================
 
 The `raw` storage format is automatically selected for all newly uploaded
-file data if no key is makred as the `default` key in the keyring. This is
+file data if no key is marked as the `default` key in the keyring. This is
 the behavior of Phabricator if you haven't configured anything.
 
 This format stores raw data without modification.
@@ -137,7 +137,7 @@ To change the format of an individual file, run this command:
 phabricator/ $ ./bin/files encode --as <format> F123 [--key <key>]
 ```
 
-This will change the storage format of the sepcified file.
+This will change the storage format of the specified file.
 
 
 Verifying Storage Formats
diff --git a/src/docs/user/configuration/managing_garbage.diviner b/src/docs/user/configuration/managing_garbage.diviner
index 1896a73904..0b18bd2a0a 100644
--- a/src/docs/user/configuration/managing_garbage.diviner
+++ b/src/docs/user/configuration/managing_garbage.diviner
@@ -21,7 +21,7 @@ Configuring Retention Policies
 
 You can reconfigure the data retention policies for most collectors.
 
-The default retention polcies should be suitable for most installs. However,
+The default retention policies should be suitable for most installs. However,
 you might want to **decrease** retention to reduce the amount of disk space
 used by some high-volume log that you don't find particularly interesting, or
 to adhere to an organizational data retention policy.
diff --git a/src/docs/user/field/conduit_changes.diviner b/src/docs/user/field/conduit_changes.diviner
index a5aa8fef6e..96a890600a 100644
--- a/src/docs/user/field/conduit_changes.diviner
+++ b/src/docs/user/field/conduit_changes.diviner
@@ -11,7 +11,7 @@ For example, when we write a new application, it usually adds several new API
 methods and may update older methods.
 
 This document discusses API stability and how to minimize disruption when
-transitionig between API versions.
+transitioning between API versions.
 
 
 Method Statuses
diff --git a/src/docs/user/field/darkconsole.diviner b/src/docs/user/field/darkconsole.diviner
index a31f0492ae..cbdfb9bda5 100644
--- a/src/docs/user/field/darkconsole.diviner
+++ b/src/docs/user/field/darkconsole.diviner
@@ -131,7 +131,7 @@ Generally, this column can help pinpoint these kinds of problems:
   - Unbatched queries which should be batched (see
     @{article:Performance: N+1 Query Problem}).
   - Opportunities to improve performance with caching.
-  - General goofiness in how service calls are woking.
+  - General goofiness in how service calls are working.
 
 If the services tab looks fine, and particularly if a page is slow but the
 "All Services" cost is small, that may indicate a problem in PHP. The best
diff --git a/src/docs/user/field/performance.diviner b/src/docs/user/field/performance.diviner
index 09413e47df..c5980acd0e 100644
--- a/src/docs/user/field/performance.diviner
+++ b/src/docs/user/field/performance.diviner
@@ -34,7 +34,7 @@ Phabricator will need substantial time to process it, it will take a long time
 to download over the network, and your browser will probably not be able to
 render it especially quickly.
 
-We may be able to improve perfomance in some cases, but Phabricator is not
+We may be able to improve performance in some cases, but Phabricator is not
 magic and can not wish away real complexity. The best solution to these problems
 is usually to find another way to solve your problem: for example, maybe the
 100MB document can be split into several smaller documents.
@@ -51,7 +51,7 @@ The upstream will be less able to help resolve unusual workloads with high
 inherent complexity, like these:
 
   - {icon times, color=red} A 100MB wiki page takes a long time to render.
-  - {icon times, color=red} A turing-complete simulation of Conway's Game of
+  - {icon times, color=red} A Turing-complete simulation of Conway's Game of
     Life implemented in 958,000 Herald rules executes slowly.
   - {icon times, color=red} Uploading an 8GB file takes several minutes.
 
diff --git a/src/docs/user/field/repository_imports.diviner b/src/docs/user/field/repository_imports.diviner
index 495758ea4a..1b1c92bbc0 100644
--- a/src/docs/user/field/repository_imports.diviner
+++ b/src/docs/user/field/repository_imports.diviner
@@ -127,7 +127,7 @@ may have briefly been down, or some configuration wasn't quite right, or the
 daemons were killed halfway through the work.
 
 These commits will retry eventually and usually succeed, but some of the retry
-time limits are very conserative (up to 24 hours) and you might not want to
+time limits are very conservative (up to 24 hours) and you might not want to
 wait that long.
 
 In the Daemon console, temporarily failures usually look like tasks in the
@@ -137,7 +137,7 @@ waiting to retry these tasks.
 
 In the daemon log, these temporary failures might have created log entries, but
 might also not have. For example, if the failure was rooted in a network issue
-that probably will create a log entry, but if the faiulre was rooted in the
+that probably will create a log entry, but if the failure was rooted in the
 daemons being abruptly killed that may not create a log entry.
 
 You can follow the instructions from "Handling Permanent Failures" above to
@@ -167,7 +167,7 @@ Forced Parsing
 ==============
 
 In rare cases, the actual tasks may be lost from the task queue. Usually, they
-have been stolen by gremlins or spritied away by ghosts, or someone may have
+have been stolen by gremlins or spirited away by ghosts, or someone may have
 been too ambitious with running manual SQL commands and deleted a bunch of
 extra things they shouldn't have.
 
@@ -226,7 +226,7 @@ General Tips
 ============
 
 Broadly, `bin/repository` contains several useful debugging commands which
-let you figure out where failures are occuring. You can add the `--trace` flag
+let you figure out where failures are occurring. You can add the `--trace` flag
 to any command to get more details about what it is doing. For any command,
 you can use `help` to learn more about what it does and which flag it takes:
 
diff --git a/src/docs/user/field/xhprof.diviner b/src/docs/user/field/xhprof.diviner
index 7d032e6865..ad6fe82cb8 100644
--- a/src/docs/user/field/xhprof.diviner
+++ b/src/docs/user/field/xhprof.diviner
@@ -88,7 +88,7 @@ that you're looking at a problem which is deeper in the stack, and you need
 to go down further to identify and understand it.
 
 Conversely, if the "Wall Time (Exclusive)" column is large, or the children
-of a call are all cheap, there's probably something expesive happening in the
+of a call are all cheap, there's probably something expensive happening in the
 call itself.
 
 The "Count" column can also sometimes tip you off that something is amiss, if
diff --git a/src/docs/user/userguide/almanac.diviner b/src/docs/user/userguide/almanac.diviner
index d6f0853430..9250724c03 100644
--- a/src/docs/user/userguide/almanac.diviner
+++ b/src/docs/user/userguide/almanac.diviner
@@ -122,7 +122,7 @@ Each service has a name, and may have properties and bindings.
 **Bindings**: Bindings are connections between services and interfaces. They
 tell callers which devices host a named service.
 
-**Networks**: Networks allow Almanac to distingiush between addresses on
+**Networks**: Networks allow Almanac to distinguish between addresses on
 different networks, like VPNs vs the public internet.
 
 If you have hosts in different VPNs or on private networks, you might have
diff --git a/src/docs/user/userguide/arcanist.diviner b/src/docs/user/userguide/arcanist.diviner
index 324ff10490..f6ccc3550f 100644
--- a/src/docs/user/userguide/arcanist.diviner
+++ b/src/docs/user/userguide/arcanist.diviner
@@ -3,7 +3,7 @@
 
 Guide to Arcanist, a command-line interface to Phabricator.
 
-Arcanists provides command-line access to many Phabricator tools (like
+Arcanist provides command-line access to many Phabricator tools (like
 Differential, Files, and Paste), integrates with static analysis ("lint") and
 unit tests, and manages common workflows like getting changes into Differential
 for review.
diff --git a/src/docs/user/userguide/arcanist_lint.diviner b/src/docs/user/userguide/arcanist_lint.diviner
index d7fcce14e8..460b8f85b6 100644
--- a/src/docs/user/userguide/arcanist_lint.diviner
+++ b/src/docs/user/userguide/arcanist_lint.diviner
@@ -169,7 +169,7 @@ Normally, you will be presented with lint messages as you are sending code for
 review. In that context, the severities behave like this:
 
   - `error` When a file contains lint errors, they are always reported. These
-    are intended to be severe problems, like a syntax error. Unresoved lint
+    are intended to be severe problems, like a syntax error. Unresolved lint
     errors require you to confirm that you want to continue.
   - `warning` When a file contains warnings, they are reported by default only
     if they appear on lines that you have changed. They are intended to be
diff --git a/src/docs/user/userguide/calendar_exports.diviner b/src/docs/user/userguide/calendar_exports.diviner
index 340cff863d..487de559a4 100644
--- a/src/docs/user/userguide/calendar_exports.diviner
+++ b/src/docs/user/userguide/calendar_exports.diviner
@@ -61,7 +61,7 @@ To export a group of events:
     - Example: All events tagged `#meetup`.
   - Select the {nav Use Results... > Export Query as .ics} action to turn
     the query into an export.
-  - Name the export with a descritive name.
+  - Name the export with a descriptive name.
   - Select a policy mode for the export (see below for discussion).
   - Click {nav Create New Export} to finish the process.
 
diff --git a/src/docs/user/userguide/calendar_imports.diviner b/src/docs/user/userguide/calendar_imports.diviner
index a4163e9bcd..a8fbc6ff09 100644
--- a/src/docs/user/userguide/calendar_imports.diviner
+++ b/src/docs/user/userguide/calendar_imports.diviner
@@ -10,7 +10,7 @@ IMPORTANT: Calendar is a prototype application. See
 @{article:User Guide: Prototype Applications}.
 
 You can import events into Phabricator to other calendar applications or from
-`.ics` files. This document will guide you through how to importe event data
+`.ics` files. This document will guide you through how to import event data
 into Phabricator.
 
 When you import events from another application, they can not be edited in
diff --git a/src/docs/user/userguide/conduit_edit.diviner b/src/docs/user/userguide/conduit_edit.diviner
index 01d15873a8..5188300e6d 100644
--- a/src/docs/user/userguide/conduit_edit.diviner
+++ b/src/docs/user/userguide/conduit_edit.diviner
@@ -37,7 +37,7 @@ Editing Objects
 
 To edit objects, pass a list of transactions and use `objectIdentifier` to
 specify which object to apply them to. You can normally pass an ID or PHID,
-and many applicaitons also allow you to pass a monogram (for example, you can
+and many applications also allow you to pass a monogram (for example, you can
 edit a task by passing `T123`).
 
 
diff --git a/src/docs/user/userguide/diffusion_api.diviner b/src/docs/user/userguide/diffusion_api.diviner
index f57d5007aa..c2508dda72 100644
--- a/src/docs/user/userguide/diffusion_api.diviner
+++ b/src/docs/user/userguide/diffusion_api.diviner
@@ -147,7 +147,7 @@ Activate the Repository
 =======================
 
 Now that any URIs have been configured, activate the repository with another
-call to `diffusion.repository.edit`. This time, modify the existing repostitory
+call to `diffusion.repository.edit`. This time, modify the existing repository
 instead of creating a new one:
 
 ```
diff --git a/src/docs/user/userguide/diffusion_hosting.diviner b/src/docs/user/userguide/diffusion_hosting.diviner
index 2443b475e8..47197f52fd 100644
--- a/src/docs/user/userguide/diffusion_hosting.diviner
+++ b/src/docs/user/userguide/diffusion_hosting.diviner
@@ -151,7 +151,7 @@ vcs-user ALL=(daemon-user) SETENV: NOPASSWD: /path/to/x, /path/to/y, ...
 
 This is just a template. In the real configuration file, you need to:
 
-  - Replace `www-user`, `dameon-user` and `vcs-user` with the correct
+  - Replace `www-user`, `daemon-user` and `vcs-user` with the correct
     usernames for your system.
   - List every binary that these users need access to, as described above.
   - Make sure each binary path is the full path to the correct binary location
diff --git a/src/docs/user/userguide/diffusion_managing.diviner b/src/docs/user/userguide/diffusion_managing.diviner
index f6ab0997c4..cc03433532 100644
--- a/src/docs/user/userguide/diffusion_managing.diviner
+++ b/src/docs/user/userguide/diffusion_managing.diviner
@@ -121,7 +121,7 @@ It is normally a good idea to leave this protection enabled because most
 scalable workflows rarely rewrite repository history and it's easy to make
 mistakes which are expensive to correct if this protection is disabled.
 
-If you do occasionally need to rewite published history, you can treat this
+If you do occasionally need to rewrite published history, you can treat this
 option like a safety: disable it, perform required rewrites, then enable it
 again.
 
@@ -148,7 +148,7 @@ Repositories can be deactivated. Deactivating a repository has these effects:
   - the repository will be hidden from view in default queries.
 
 When repositories are created for the first time, they are deactivated. This
-gives you an opportuinty to customize settings, like adjusting policies or
+gives you an opportunity to customize settings, like adjusting policies or
 configuring a URI to observe. You must activate a repository before it will
 start working normally.
 
@@ -177,7 +177,7 @@ The **Policies** section of the management interface allows you to review and
 manage repository access policies.
 
 You can configure granular access policies for each repository to control who
-can view, clone, administate, and push to the repository.
+can view, clone, administrate, and push to the repository.
 
 
 Policies: View
@@ -302,7 +302,7 @@ Actions
 The **Actions** panel can configure notifications and publishing behavior.
 
 Normally, Phabricator publishes notifications when it discovers new commits.
-You can disable publishing for a repository by turning off **Publish/Noitfy**.
+You can disable publishing for a repository by turning off **Publish/Notify**.
 This will disable notifications, feed, and Herald (including audits and build
 plans) for this repository.
 
@@ -347,7 +347,7 @@ Commit Identifiers
 ==================
 
 Diffusion uses repository identifiers and information about the commit itself
-to generate globally unique identifers for each commit, like `rE12345`.
+to generate globally unique identifiers for each commit, like `rE12345`.
 
 Each commit may have several identifiers:
 
diff --git a/src/docs/user/userguide/diffusion_uris.diviner b/src/docs/user/userguide/diffusion_uris.diviner
index d1f4b541bc..140948f55e 100644
--- a/src/docs/user/userguide/diffusion_uris.diviner
+++ b/src/docs/user/userguide/diffusion_uris.diviner
@@ -20,7 +20,7 @@ is hosted elsewhere (like GitHub or Bitbucket) and track updates to the remote
 repository. This will create a read-only copy of the repository in Phabricator.
 
 **Mirror Repositories**: Phabricator can publish any repository to mirrors,
-overwiting them with an exact copy of the repository that stays up to date as
+overwriting them with an exact copy of the repository that stays up to date as
 the source changes. This works with both local repositories that Phabricator is
 hosting and remote repositories that Phabricator is observing.
 
@@ -293,7 +293,7 @@ For builtin URIs in //Read Only// or //Read/Write// mode, the most
 human-readable URI defaults to //Always// and the others default to //Never//.
 
 **Always**: This URI will be shown to users as a clone/checkout URI. You can
-add URIs in this mode to cutomize exactly what users are shown.
+add URIs in this mode to customize exactly what users are shown.
 
 **Never**: This URI will not be shown to users. You can hide less-preferred
 URIs to guide users to the URIs they should be using to interact with the
diff --git a/src/docs/user/userguide/drydock.diviner b/src/docs/user/userguide/drydock.diviner
index 44a645bf8a..0d43f7f3f0 100644
--- a/src/docs/user/userguide/drydock.diviner
+++ b/src/docs/user/userguide/drydock.diviner
@@ -151,7 +151,7 @@ thousands of hosts in hundreds of pools, and far beyond that with a little
 work.
 
 Drydock is intended to solve resource management problems at very large scales
-and minimzes blocking operations, locks, and artificial sequencing. Drydock is
+and minimizes blocking operations, locks, and artificial sequencing. Drydock is
 designed to fully utilize an almost arbitrarily large pool of resources and
 improve performance roughly linearly with available hardware.
 
diff --git a/src/docs/user/userguide/drydock_blueprints.diviner b/src/docs/user/userguide/drydock_blueprints.diviner
index 21c4342fa6..c150c503d9 100644
--- a/src/docs/user/userguide/drydock_blueprints.diviner
+++ b/src/docs/user/userguide/drydock_blueprints.diviner
@@ -47,7 +47,7 @@ operations on trusted hosts. For additional discussion, see
 
 This also broadly prevents Drydock from surprising you by coming up with a
 valid but unintended solution to an allocation problem which runs some
-operation on resources that are techincally suitable but not desirable. For
+operation on resources that are technically suitable but not desirable. For
 example, you may not want your Android builds running on your iPhone build
 tier, even if there's no technical reason they can't.
 
diff --git a/src/docs/user/userguide/drydock_security.diviner b/src/docs/user/userguide/drydock_security.diviner
index f12586ab35..9a212437c7 100644
--- a/src/docs/user/userguide/drydock_security.diviner
+++ b/src/docs/user/userguide/drydock_security.diviner
@@ -141,7 +141,7 @@ host which is not on a privileged subnet. For example, use a
 Even if the host is not privileged, many Drydock processes have some level of
 privilege (enabling them to clone repositories, or report test results back to
 Phabricator). Be aware that tests can hijack credentials they are run with,
-and potentialy hijack credentials given to other processes on the same hosts.
+and potentially hijack credentials given to other processes on the same hosts.
 You should use credentials with a minimum set of privileges and assume all
 processes on a host have the highest level of access that any process on the
 host has.
diff --git a/src/docs/user/userguide/forms.diviner b/src/docs/user/userguide/forms.diviner
index 98233e4931..034293ff29 100644
--- a/src/docs/user/userguide/forms.diviner
+++ b/src/docs/user/userguide/forms.diviner
@@ -34,7 +34,7 @@ objects, which will also affect their ability to take inline actions in the
 comment form if you're working in an application which supports comments. This
 can streamline the edit workflow for less experienced users.
 
-Anyone can use prefiling, but you must have permission to configure an
+Anyone can use prefilling, but you must have permission to configure an
 application in order to modify the application's forms. By default, only
 administrators can configure applications.
 
diff --git a/src/docs/user/userguide/herald.diviner b/src/docs/user/userguide/herald.diviner
index 6c5c6239cb..c5a008474d 100644
--- a/src/docs/user/userguide/herald.diviner
+++ b/src/docs/user/userguide/herald.diviner
@@ -126,7 +126,7 @@ match the content. You can use these together to express conditions like
 For example, if you want to match revisions which add or remove calls to a
 "muffinize" function, //but only in JS files//, you can set the value to
 `["/\\.js$/", "/muffinize/"]` or similar. This condition is satisfied only
-when the filename matches the first expression and the conent matches the
+when the filename matches the first expression and the content matches the
 second expression.
 
 **Another Herald rule**: you can create Herald rules which depend on other
diff --git a/src/docs/user/userguide/jump.diviner b/src/docs/user/userguide/jump.diviner
index 413c4fa47b..0421f194a8 100644
--- a/src/docs/user/userguide/jump.diviner
+++ b/src/docs/user/userguide/jump.diviner
@@ -1,7 +1,7 @@
 @title Search User Guide: Shortcuts
 @group userguide
 
-Command reference for global search shorcuts.
+Command reference for global search shortcuts.
 
 Overview
 ========
diff --git a/src/docs/user/userguide/owners.diviner b/src/docs/user/userguide/owners.diviner
index a9a52bfab8..f30fe5023c 100644
--- a/src/docs/user/userguide/owners.diviner
+++ b/src/docs/user/userguide/owners.diviner
@@ -24,7 +24,7 @@ including these paths:
 
 Any files in those directories are considered to be part of the package, and
 you can now conveniently refer to them (for example, in a Herald rule) by
-refering to the package instead of copy/pasting a huge regular expression
+referring to the package instead of copy/pasting a huge regular expression
 into a bunch of places.
 
 If new source files are later added, or the scope of the package otherwise
diff --git a/src/docs/user/userguide/projects.diviner b/src/docs/user/userguide/projects.diviner
index 573c8c4c69..4e3ab3616f 100644
--- a/src/docs/user/userguide/projects.diviner
+++ b/src/docs/user/userguide/projects.diviner
@@ -14,7 +14,7 @@ a name and an icon, and may optionally have members.
 
 For example, you can create projects to provide:
 
-  - **Organization**: Create a project to represent a product or initative,
+  - **Organization**: Create a project to represent a product or initiative,
     then use it to organize related work.
   - **Groups**: Create a project to represent a group of people (like a team),
     then add members of the group as project members.
diff --git a/src/docs/user/userguide/remarkup.diviner b/src/docs/user/userguide/remarkup.diviner
index 6002867b78..28a1a31fc4 100644
--- a/src/docs/user/userguide/remarkup.diviner
+++ b/src/docs/user/userguide/remarkup.diviner
@@ -366,7 +366,7 @@ can be found on the date stamp of any transaction/comment):
 
   T123#412       # Link to comment id #412 of task T123
 
-See the Phabricator configuraton setting `remarkup.ignored-object-names` to
+See the Phabricator configuration setting `remarkup.ignored-object-names` to
 modify this behavior.
 
 = Embedding Objects
diff --git a/src/docs/user/userguide/search.diviner b/src/docs/user/userguide/search.diviner
index 875edc6d0f..d1c6182128 100644
--- a/src/docs/user/userguide/search.diviner
+++ b/src/docs/user/userguide/search.diviner
@@ -87,7 +87,7 @@ view show the results you most often want.
 
 You can share queries with other users by sending them the URL. This will run
 the same query for them with all the parameters you've set (they may see
-different results than you do, because they may not have the same permisions).
+different results than you do, because they may not have the same permissions).
 
 
 Typeaheads
diff --git a/src/docs/user/userguide/spaces.diviner b/src/docs/user/userguide/spaces.diviner
index 212def61f3..4a748fb25b 100644
--- a/src/docs/user/userguide/spaces.diviner
+++ b/src/docs/user/userguide/spaces.diviner
@@ -70,7 +70,7 @@ spaces exist. This simplifies the UI for users with limited access.
 Space Policies
 ==============
 
-Briefly, spacess affect policies like this:
+Briefly, spaces affect policies like this:
 
   - Spaces apply their view policy to all objects inside the space.
   - Space policies are absolute, and stronger than all other policies. A
diff --git a/src/docs/user/userguide/users.diviner b/src/docs/user/userguide/users.diviner
index e96bf8f354..d66f8080d3 100644
--- a/src/docs/user/userguide/users.diviner
+++ b/src/docs/user/userguide/users.diviner
@@ -73,7 +73,7 @@ The **Mailing List** role for an account can not be changed after the account
 is created.
 
 Some options can be configured for mailing lists by browsing to the list user's
-profile and clicking {nav Edit Settings}. You can change the addresss for a
+profile and clicking {nav Edit Settings}. You can change the address for a
 list by editing "Email Addresses" here, choose the language and format for
 email the list receives, and customize which actions the list is notified about.
 
diff --git a/src/infrastructure/customfield/field/PhabricatorCustomFieldList.php b/src/infrastructure/customfield/field/PhabricatorCustomFieldList.php
index cb1e56711d..efe12e68b8 100644
--- a/src/infrastructure/customfield/field/PhabricatorCustomFieldList.php
+++ b/src/infrastructure/customfield/field/PhabricatorCustomFieldList.php
@@ -160,7 +160,7 @@ final class PhabricatorCustomFieldList extends Phobject {
       if ($value !== null) {
         switch ($field->getStyleForPropertyView()) {
           case 'header':
-            // We want to hide headers if the fields the're assciated with
+            // We want to hide headers if the fields they're associated with
             // don't actually produce any visible properties. For example, in a
             // list like this:
             //
diff --git a/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php b/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php
index c569f0c695..a8dc5061e7 100644
--- a/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php
+++ b/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php
@@ -5,7 +5,7 @@ final class PhabricatorWorkerTriggerQuery
 
   // NOTE: This is a PolicyAware query so it can work with other infrastructure
   // like handles; triggers themselves are low-level and do not have
-  // meaninguful policies.
+  // meaningful policies.
 
   const ORDER_ID = 'id';
   const ORDER_EXECUTION = 'execution';
@@ -69,7 +69,7 @@ final class PhabricatorWorkerTriggerQuery
 
   protected function nextPage(array $page) {
     // NOTE: We don't implement paging because we don't currently ever need
-    // it and paging ORDER_EXCUTION is a hassle.
+    // it and paging ORDER_EXECUTION is a hassle.
     throw new PhutilMethodNotImplementedException();
   }
 
diff --git a/src/infrastructure/diff/view/PHUIDiffGraphView.php b/src/infrastructure/diff/view/PHUIDiffGraphView.php
index 76ae0b2045..85741faab7 100644
--- a/src/infrastructure/diff/view/PHUIDiffGraphView.php
+++ b/src/infrastructure/diff/view/PHUIDiffGraphView.php
@@ -160,7 +160,7 @@ final class PHUIDiffGraphView extends Phobject {
 
             $terminated[$ii] = true;
 
-            // If this thread is joinining some other node here, we don't want
+            // If this thread is joining some other node here, we don't want
             // to terminate it.
             if (isset($graph[$key + 1])) {
               $joins = $graph[$key + 1]['join'];
diff --git a/src/infrastructure/diff/view/PHUIDiffInlineCommentTableScaffold.php b/src/infrastructure/diff/view/PHUIDiffInlineCommentTableScaffold.php
index 5da25389a5..94eb91ec58 100644
--- a/src/infrastructure/diff/view/PHUIDiffInlineCommentTableScaffold.php
+++ b/src/infrastructure/diff/view/PHUIDiffInlineCommentTableScaffold.php
@@ -4,7 +4,7 @@
  * Wraps an inline comment row scaffold in a table.
  *
  * This scaffold is used to ship inlines over the wire to the client, so they
- * arrive in a form that's easy to mainipulate (a valid table node).
+ * arrive in a form that's easy to manipulate (a valid table node).
  */
 final class PHUIDiffInlineCommentTableScaffold extends AphrontView {
 
diff --git a/src/infrastructure/env/PhabricatorEnv.php b/src/infrastructure/env/PhabricatorEnv.php
index 70ddb80630..ba05d0409d 100644
--- a/src/infrastructure/env/PhabricatorEnv.php
+++ b/src/infrastructure/env/PhabricatorEnv.php
@@ -670,7 +670,7 @@ final class PhabricatorEnv extends Phobject {
    * Detect if a URI identifies some valid linkable remote resource.
    *
    * @param string URI to test.
-   * @return bool True if a URI idenfies a remote resource with an allowed
+   * @return bool True if a URI identifies a remote resource with an allowed
    *              protocol.
    * @task uri
    */
diff --git a/src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php b/src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php
index 95ba57f0c0..184c2d5151 100644
--- a/src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php
+++ b/src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php
@@ -730,7 +730,7 @@ final class PhabricatorUSEnglishTranslation
       '%s added %s reverting commit(s) for %s: %s.' => array(
         array(
           '%s added a reverting commit for %3$s: %4$s.',
-          '%s added reverting commitsi for %3$s: %4$s.',
+          '%s added reverting commits for %3$s: %4$s.',
         ),
       ),
 
@@ -1564,7 +1564,7 @@ final class PhabricatorUSEnglishTranslation
       '%s uninvited %s attendee(s): %s.' =>
         '%s uninvited: %3$s.',
 
-      '%s invited %s attendee(s): %s; uninvinted %s attendee(s): %s.' =>
+      '%s invited %s attendee(s): %s; uninvited %s attendee(s): %s.' =>
         '%s invited: %3$s; uninvited: %5$s.',
 
       '%s invited %s attendee(s) to %s: %s.' =>
@@ -1573,7 +1573,7 @@ final class PhabricatorUSEnglishTranslation
       '%s uninvited %s attendee(s) to %s: %s.' =>
         '%s removed invites for %3$s: %4$s.',
 
-      '%s updated the invite list for %s, invited %s: %s; uninvinted %s: %s.' =>
+      '%s updated the invite list for %s, invited %s: %s; uninvited %s: %s.' =>
         '%s updated the invite list for %s, invited: %4$s; uninvited: %6$s.',
 
       'Restart %s build(s)?' => array(
diff --git a/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php b/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php
index c2b8ce04e0..9bebfe6957 100644
--- a/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php
+++ b/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php
@@ -175,7 +175,7 @@ abstract class PhabricatorCursorPagedPolicyAwareQuery
    * So, generally, internal paging must bypass policy controls.
    *
    * This method returns the appropriate viewer, based on the context in which
-   * the paging is occuring.
+   * the paging is occurring.
    *
    * @return PhabricatorUser Viewer for executing paging queries.
    */
@@ -536,7 +536,7 @@ abstract class PhabricatorCursorPagedPolicyAwareQuery
    *
    * @param AphrontDatabaseConnection Connection query will execute on.
    * @param list<map> Column description dictionaries.
-   * @param map Additional constuction options.
+   * @param map Additional construction options.
    * @return string Query clause.
    * @task paging
    */
diff --git a/src/infrastructure/storage/lisk/LiskDAO.php b/src/infrastructure/storage/lisk/LiskDAO.php
index 94eef7d163..0c940ef5f0 100644
--- a/src/infrastructure/storage/lisk/LiskDAO.php
+++ b/src/infrastructure/storage/lisk/LiskDAO.php
@@ -1121,7 +1121,7 @@ abstract class LiskDAO extends Phobject {
     $this->willSaveObject();
     $data = $this->getAllLiskPropertyValues();
 
-    // Remove colums flagged as nonmutable from the update statement.
+    // Remove columns flagged as nonmutable from the update statement.
     $no_mutate = $this->getConfigOption(self::CONFIG_NO_MUTATE);
     if ($no_mutate) {
       foreach ($no_mutate as $column) {
diff --git a/src/infrastructure/storage/lisk/__tests__/LiskFixtureTestCase.php b/src/infrastructure/storage/lisk/__tests__/LiskFixtureTestCase.php
index 065c23e2ff..92c43fcd4b 100644
--- a/src/infrastructure/storage/lisk/__tests__/LiskFixtureTestCase.php
+++ b/src/infrastructure/storage/lisk/__tests__/LiskFixtureTestCase.php
@@ -98,7 +98,7 @@ final class LiskFixtureTestCase extends PhabricatorTestCase {
     $obj = new HarbormasterObject();
     $conn_w = $obj->establishConnection('w');
 
-    // Test that the counter bascially behaves as expected.
+    // Test that the counter basically behaves as expected.
     $this->assertEqual(1, LiskDAO::loadNextCounterValue($conn_w, 'a'));
     $this->assertEqual(2, LiskDAO::loadNextCounterValue($conn_w, 'a'));
     $this->assertEqual(3, LiskDAO::loadNextCounterValue($conn_w, 'a'));
diff --git a/src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php b/src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php
index a03ef41c4d..f6b120ba83 100644
--- a/src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php
+++ b/src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php
@@ -181,8 +181,8 @@ abstract class PhabricatorStorageManagementWorkflow
     if (!$this->force && !$api->isCharacterSetAvailable('utf8mb4')) {
       $message = pht(
         "You have an old version of MySQL (older than 5.5) which does not ".
-        "support the utf8mb4 character set. We strongly recomend upgrading to ".
-        "5.5 or newer.\n\n".
+        "support the utf8mb4 character set. We strongly recommend upgrading ".
+        "to 5.5 or newer.\n\n".
         "If you apply adjustments now and later update MySQL to 5.5 or newer, ".
         "you'll need to apply adjustments again (and they will take a long ".
         "time).\n\n".
diff --git a/src/infrastructure/util/PhabricatorHash.php b/src/infrastructure/util/PhabricatorHash.php
index 19c8414539..5ca4e9b53c 100644
--- a/src/infrastructure/util/PhabricatorHash.php
+++ b/src/infrastructure/util/PhabricatorHash.php
@@ -11,7 +11,7 @@ final class PhabricatorHash extends Phobject {
    * weak. Callers should prefer @{method:digestWithNamedKey}.
    *
    * @param   string  Input string.
-   * @return  string  32-byte hexidecimal SHA1+HMAC hash.
+   * @return  string  32-byte hexadecimal SHA1+HMAC hash.
    */
   public static function weakDigest($string, $key = null) {
     if ($key === null) {
diff --git a/src/infrastructure/util/password/PhabricatorPasswordHasher.php b/src/infrastructure/util/password/PhabricatorPasswordHasher.php
index 7972acc0a8..f0f30045c0 100644
--- a/src/infrastructure/util/password/PhabricatorPasswordHasher.php
+++ b/src/infrastructure/util/password/PhabricatorPasswordHasher.php
@@ -70,7 +70,7 @@ abstract class PhabricatorPasswordHasher extends Phobject {
 
   /**
    * Return an indicator of this hasher's strength. When choosing to hash
-   * new passwords, the strongest available hasher which is usuable for new
+   * new passwords, the strongest available hasher which is usable for new
    * passwords will be used, and the presence of a stronger hasher will
    * prompt users to update their hashes.
    *
@@ -208,7 +208,7 @@ abstract class PhabricatorPasswordHasher extends Phobject {
    * Get all available password hashers. This may include hashers which can not
    * actually be used (for example, a required extension is missing).
    *
-   * @return list<PhabicatorPasswordHasher> Hasher objects.
+   * @return list<PhabricatorPasswordHasher> Hasher objects.
    * @task hashing
    */
   public static function getAllHashers() {
@@ -243,7 +243,7 @@ abstract class PhabricatorPasswordHasher extends Phobject {
    * Get all usable password hashers. This may include hashers which are
    * not desirable or advisable.
    *
-   * @return list<PhabicatorPasswordHasher> Hasher objects.
+   * @return list<PhabricatorPasswordHasher> Hasher objects.
    * @task hashing
    */
   public static function getAllUsableHashers() {
@@ -280,7 +280,7 @@ abstract class PhabricatorPasswordHasher extends Phobject {
 
 
   /**
-   * Get the hashser for a given stored hash.
+   * Get the hasher for a given stored hash.
    *
    * @return PhabricatorPasswordHasher Corresponding hasher.
    * @task hashing
diff --git a/src/view/AphrontView.php b/src/view/AphrontView.php
index c014d7d50c..669742f29b 100644
--- a/src/view/AphrontView.php
+++ b/src/view/AphrontView.php
@@ -42,7 +42,7 @@ abstract class AphrontView extends Phobject
 
 
   /**
-   * Test if a viewer has been set on this elmeent.
+   * Test if a viewer has been set on this element.
    *
    * @return bool True if a viewer is available.
    */
diff --git a/src/view/page/PhabricatorStandardPageView.php b/src/view/page/PhabricatorStandardPageView.php
index d26cca945f..78ff716a44 100644
--- a/src/view/page/PhabricatorStandardPageView.php
+++ b/src/view/page/PhabricatorStandardPageView.php
@@ -272,7 +272,7 @@ final class PhabricatorStandardPageView extends PhabricatorBarePageView
               'doc_href' => $doc_href,
               'message' => pht(
                 'Phabricator thinks you are using %s, but your '.
-                'client is conviced that it is using %s. This is a serious '.
+                'client is convinced that it is using %s. This is a serious '.
                 'misconfiguration with subtle, but significant, consequences.',
                 $server_protocol, $client_protocol),
             ));
diff --git a/src/view/phui/PHUIHeaderView.php b/src/view/phui/PHUIHeaderView.php
index dc836f3d3a..53ec096265 100644
--- a/src/view/phui/PHUIHeaderView.php
+++ b/src/view/phui/PHUIHeaderView.php
@@ -166,7 +166,7 @@ final class PHUIHeaderView extends AphrontTagView {
     $classes[] = 'phui-header-shell';
 
     if ($this->noBackground) {
-      $classes[] = 'phui-header-no-backgound';
+      $classes[] = 'phui-header-no-background';
     }
 
     if ($this->bleedHeader) {
diff --git a/webroot/rsrc/css/phui/phui-document.css b/webroot/rsrc/css/phui/phui-document.css
index e5985f8889..954a220229 100644
--- a/webroot/rsrc/css/phui/phui-document.css
+++ b/webroot/rsrc/css/phui/phui-document.css
@@ -35,13 +35,13 @@
 }
 
 .phui-document-content
-  .phui-header-shell.phui-header-no-backgound {
+  .phui-header-shell.phui-header-no-background {
   border-bottom: 1px solid {$thinblueborder};
   margin: 0 0 16px 0;
 }
 
 .phui-document-content
-  .phui-header-shell.phui-header-no-backgound
+  .phui-header-shell.phui-header-no-background
   .phui-header-view {
   padding: 8px 0 4px;
 }
diff --git a/webroot/rsrc/css/phui/phui-header-view.css b/webroot/rsrc/css/phui/phui-header-view.css
index 18b1464e53..478dde2153 100644
--- a/webroot/rsrc/css/phui/phui-header-view.css
+++ b/webroot/rsrc/css/phui/phui-header-view.css
@@ -34,7 +34,7 @@
   vertical-align: middle;
 }
 
-body .phui-header-shell.phui-header-no-backgound {
+body .phui-header-shell.phui-header-no-background {
   background-color: transparent;
   border: none;
 }
diff --git a/webroot/rsrc/js/application/conpherence/behavior-menu.js b/webroot/rsrc/js/application/conpherence/behavior-menu.js
index a5566dc5a4..43488ce4ff 100644
--- a/webroot/rsrc/js/application/conpherence/behavior-menu.js
+++ b/webroot/rsrc/js/application/conpherence/behavior-menu.js
@@ -94,7 +94,7 @@ JX.behavior('conpherence-menu', function(config) {
   var _oldDevice = null;
 
   /**
-   * Initializes this behavior based on all the configuraton jonx and the
+   * Initializes this behavior based on all the configuration jonx and the
    * result of JX.Device.getDevice();
    */
   function init() {
diff --git a/webroot/rsrc/js/application/conpherence/behavior-participant-pane.js b/webroot/rsrc/js/application/conpherence/behavior-participant-pane.js
index 8c9fc056f4..e428cbdbce 100644
--- a/webroot/rsrc/js/application/conpherence/behavior-participant-pane.js
+++ b/webroot/rsrc/js/application/conpherence/behavior-participant-pane.js
@@ -69,7 +69,7 @@ JX.behavior('conpherence-participant-pane', function() {
       // While the user is removing themselves, disable the notification
       // update behavior. If we don't do this, the user can get an error
       // when they remove themselves about permissions as the notification
-      // code tries to load what jist happened.
+      // code tries to load what just happened.
       var loadedPhid = threadManager.getLoadedThreadPHID();
       threadManager.setLoadedThreadPHID(null);
 
diff --git a/webroot/rsrc/js/application/diffusion/DiffusionLocateFileSource.js b/webroot/rsrc/js/application/diffusion/DiffusionLocateFileSource.js
index 14894f6cc0..1f94f349d3 100644
--- a/webroot/rsrc/js/application/diffusion/DiffusionLocateFileSource.js
+++ b/webroot/rsrc/js/application/diffusion/DiffusionLocateFileSource.js
@@ -260,7 +260,7 @@ JX.install('DiffusionLocateFileSource', {
 
     /**
      * Score a matching string by finding the longest prefix of the search
-     * query it contains continguously.
+     * query it contains contiguously.
      */
     scoreMatch: function(haystack, haypos, search) {
       var pos = 0;
diff --git a/webroot/rsrc/js/application/pholio/behavior-pholio-mock-view.js b/webroot/rsrc/js/application/pholio/behavior-pholio-mock-view.js
index 3991f4cc4e..0a02aebce7 100644
--- a/webroot/rsrc/js/application/pholio/behavior-pholio-mock-view.js
+++ b/webroot/rsrc/js/application/pholio/behavior-pholio-mock-view.js
@@ -201,7 +201,7 @@ JX.behavior('pholio-mock-view', function(config, statics) {
   }
 
   function render_image_header(image) {
-    // Render image dimensions and visible size. If we have this infomation
+    // Render image dimensions and visible size. If we have this information
     // from the server we can display some of it immediately; otherwise, we need
     // to wait for the image to load so we can read dimension information from
     // it.
diff --git a/webroot/rsrc/js/core/Prefab.js b/webroot/rsrc/js/core/Prefab.js
index c5a8ef5e9d..979ad3473b 100644
--- a/webroot/rsrc/js/core/Prefab.js
+++ b/webroot/rsrc/js/core/Prefab.js
@@ -184,7 +184,7 @@ JX.install('Prefab', {
       var self_hits = {};
 
       // We'll put matches where the user's input is a prefix of the name
-      // above mathches where that isn't true.
+      // above matches where that isn't true.
       var prefix_hits = {};
 
       var tokens = this.tokenize(value);
@@ -212,7 +212,7 @@ JX.install('Prefab', {
         }
 
         // If one result is open and one is closed, show the open result
-        // first. The "!" tricks here are becaused closed values are display
+        // first. The "!" tricks here are because closed values are display
         // strings, so the value is either `null` or some truthy string. If
         // we compare the values directly, we'll apply this rule to two
         // objects which are both closed but for different reasons, like
diff --git a/webroot/rsrc/js/core/behavior-device.js b/webroot/rsrc/js/core/behavior-device.js
index d74939a7e0..d68a46ff4e 100644
--- a/webroot/rsrc/js/core/behavior-device.js
+++ b/webroot/rsrc/js/core/behavior-device.js
@@ -27,7 +27,7 @@ JX.install('Device', {
       var self = JX.Device;
 
       // Even when we emit a '<meta name="viewport" ... />' tag which tells
-      // devices to fit the conent to the screen width, we'll sometimes measure
+      // devices to fit the content to the screen width, we'll sometimes measure
       // a viewport dimension which is larger than the available screen width,
       // particularly if we check too early.
 
diff --git a/webroot/rsrc/js/phuix/PHUIXAutocomplete.js b/webroot/rsrc/js/phuix/PHUIXAutocomplete.js
index df4a031377..a7f719dab4 100644
--- a/webroot/rsrc/js/phuix/PHUIXAutocomplete.js
+++ b/webroot/rsrc/js/phuix/PHUIXAutocomplete.js
@@ -543,7 +543,7 @@ JX.install('PHUIXAutocomplete', {
 
       // If the input is terminated by a space or another word-terminating
       // punctuation mark, we're going to deactivate if the results can not
-      // be refined by addding more words.
+      // be refined by adding more words.
 
       // The idea is that if you type "@alan ab", you're allowed to keep
       // editing "ab" until you type a space, period, or other terminator,
diff --git a/webroot/rsrc/js/phuix/PHUIXDropdownMenu.js b/webroot/rsrc/js/phuix/PHUIXDropdownMenu.js
index 3f93c1ee41..d9d86bf595 100644
--- a/webroot/rsrc/js/phuix/PHUIXDropdownMenu.js
+++ b/webroot/rsrc/js/phuix/PHUIXDropdownMenu.js
@@ -124,7 +124,7 @@ JX.install('PHUIXDropdownMenu', {
       }
 
       // If this action was built dynamically with PHUIXActionView, don't
-      // do anything by default. The caller is repsonsible for installing a
+      // do anything by default. The caller is responsible for installing a
       // handler if they want to react to clicks.
       if (e.getNode('phuix-action-view')) {
         return;