Merge ~cjwatson/launchpad:codespell-testing-translations-xmlrpc into launchpad:master

[Colin Watson]

Fix spelling errors in lp.{testing,translations,xmlrpc}

[Colin Watson (cjwatson)]

review: Approve

diff --git a/lib/lp/testing/factory.py b/lib/lp/testing/factory.py
index 54ab372..098b17f 100644
--- a/lib/lp/testing/factory.py
+++ b/lib/lp/testing/factory.py
@@ -676,7 +676,7 @@ class LaunchpadObjectFactory(ObjectFactory):
         del naked_person
 
         if selfgenerated_bugnotifications:
-            # Set it explicitely only when True because the default
+            # Set it explicitly only when True because the default
             # is False.
             person.selfgenerated_bugnotifications = True
 
@@ -2940,13 +2940,13 @@ class LaunchpadObjectFactory(ObjectFactory):
         )
 
     def makePackageCodeImport(self, sourcepackage=None, **kwargs):
-        """Make a code import targetting a sourcepackage."""
+        """Make a code import targeting a sourcepackage."""
         if sourcepackage is None:
             sourcepackage = self.makeSourcePackage()
         return self.makeCodeImport(context=sourcepackage, **kwargs)
 
     def makeProductCodeImport(self, product=None, **kwargs):
-        """Make a code import targetting a product."""
+        """Make a code import targeting a product."""
         if product is None:
             product = self.makeProduct()
         return self.makeCodeImport(context=product, **kwargs)
diff --git a/lib/lp/testing/layers.py b/lib/lp/testing/layers.py
index 1e27824..df1c60c 100644
--- a/lib/lp/testing/layers.py
+++ b/lib/lp/testing/layers.py
@@ -762,7 +762,7 @@ class DatabaseLayer(BaseLayer):
             )
         # Reset/bring up the db - makes it available for either the next test,
         # or a subordinate layer which builds on the db. This wastes one setup
-        # per db layer teardown per run, but thats tolerable.
+        # per db layer teardown per run, but that's tolerable.
         cls._db_fixture.setUp()
 
     @classmethod
diff --git a/lib/lp/testing/matchers.py b/lib/lp/testing/matchers.py
index 47e7c92..ad089c0 100644
--- a/lib/lp/testing/matchers.py
+++ b/lib/lp/testing/matchers.py
@@ -59,9 +59,9 @@ class BrowsesWithQueryLimit(Matcher):
     def __init__(self, query_limit, user, view_name=None, **options):
         """Create a BrowsesWithQueryLimit checking for limit query_limit.
 
-        :param query_limit: The number of queries permited for the page.
+        :param query_limit: The number of queries permitted for the page.
         :param user: The user to use to render the page.
-        :param view_name: The name of the view to use to render tha page.
+        :param view_name: The name of the view to use to render the page.
         :param options: Additional options for view generation eg rootsite.
         """
         self.query_limit = query_limit
@@ -442,7 +442,7 @@ class MatchesTagText(Matcher):
         return "matches widget %r text" % self.tag_id
 
     def match(self, matchee):
-        # Here to avoid circular dependancies.
+        # Here to avoid circular dependencies.
         from lp.testing.pages import extract_text
 
         widgets = self.soup_content.find_all(id=self.tag_id)
@@ -467,7 +467,7 @@ class MatchesPickerText(Matcher):
         return "matches widget %r text" % self.widget_id
 
     def match(self, matchee):
-        # Here to avoid circular dependancies.
+        # Here to avoid circular dependencies.
         from lp.testing.pages import extract_text
 
         widgets = self.soup_content.find_all(id=self.widget_id)
diff --git a/lib/lp/testing/pages.py b/lib/lp/testing/pages.py
index 0a032f4..c01071d 100644
--- a/lib/lp/testing/pages.py
+++ b/lib/lp/testing/pages.py
@@ -1010,7 +1010,7 @@ def setUpGlobs(test):
     )
 
     test.globs["ANONYMOUS"] = ANONYMOUS
-    # If a unicode URL is opened by the test browswer, later navigation
+    # If a unicode URL is opened by the test browser, later navigation
     # raises ValueError exceptions in /usr/lib/python2.4/Cookie.py
     test.globs["canonical_url"] = safe_canonical_url
     test.globs["factory"] = LaunchpadObjectFactory()
diff --git a/lib/lp/testing/tests/test_fixture.py b/lib/lp/testing/tests/test_fixture.py
index 1fc6328..d258341 100644
--- a/lib/lp/testing/tests/test_fixture.py
+++ b/lib/lp/testing/tests/test_fixture.py
@@ -175,7 +175,7 @@ class TestPGBouncerFixtureWithCA(TestCase):
         with PGBouncerFixture() as pgbouncer:
             assert self.is_connected()
 
-            # pgbouncer is transparant. To confirm we are connecting via
+            # pgbouncer is transparent. To confirm we are connecting via
             # pgbouncer, we need to shut it down and confirm our
             # connections are dropped.
             pgbouncer.stop()
@@ -196,7 +196,7 @@ class TestPGBouncerFixtureWithCA(TestCase):
         with PGBouncerFixture() as pgbouncer:
             assert self.is_connected()
 
-            # pgbouncer is transparant. To confirm we are connecting via
+            # pgbouncer is transparent. To confirm we are connecting via
             # pgbouncer, we need to shut it down and confirm our
             # connections are dropped.
             pgbouncer.stop()
diff --git a/lib/lp/testing/tests/test_layers_functional.py b/lib/lp/testing/tests/test_layers_functional.py
index 84bca68..88a9079 100644
--- a/lib/lp/testing/tests/test_layers_functional.py
+++ b/lib/lp/testing/tests/test_layers_functional.py
@@ -318,7 +318,7 @@ class LibrarianLayerTest(TestCase, TestWithFixtures):
 
 
 class LibrarianResetTestCase(TestCase):
-    """Our page tests need to run multple tests without destroying
+    """Our page tests need to run multiple tests without destroying
     the librarian database in between.
     """
 
diff --git a/lib/lp/testing/tests/test_login.py b/lib/lp/testing/tests/test_login.py
index 3cc8464..4824a77 100644
--- a/lib/lp/testing/tests/test_login.py
+++ b/lib/lp/testing/tests/test_login.py
@@ -220,7 +220,7 @@ class TestLoginHelpers(TestCaseWithFactory):
         self.assertLoggedOut()
 
     def test_person_logged_in_restores_person_even_when_raises(self):
-        # Once outside of the person_logged_in context, the originially
+        # Once outside of the person_logged_in context, the originally
         # logged-in person is re-logged in.
         a = self.factory.makePerson()
         login_as(a)
diff --git a/lib/lp/testing/tests/test_testcase.py b/lib/lp/testing/tests/test_testcase.py
index a32d16c..9031f30 100644
--- a/lib/lp/testing/tests/test_testcase.py
+++ b/lib/lp/testing/tests/test_testcase.py
@@ -87,7 +87,7 @@ class TestCaptureOops(TestCaseWithFactory):
         content = io.BytesIO()
         content.writelines(self.getDetails()["oops-0"].iter_bytes())
         content.seek(0)
-        # Safety net: ensure that no autocasts have occured even on Python 2.6
+        # Safety net: ensure that no autocasts have occurred even on Python 2.6
         # which is slightly better.
         self.assertIsInstance(content.getvalue(), bytes)
         # In tests it should be rfc822 for easy reading.
diff --git a/lib/lp/testing/yuixhr.py b/lib/lp/testing/yuixhr.py
index 746105a..683a14f 100644
--- a/lib/lp/testing/yuixhr.py
+++ b/lib/lp/testing/yuixhr.py
@@ -264,7 +264,7 @@ class YUITestFixtureControllerView(LaunchpadView):
             return
         path, ext = os.path.splitext(self.traversed_path)
         # We need to route requests with build/js in them to the combo loader
-        # js files so we can load those to bootstap.
+        # js files so we can load those to bootstrap.
         if path.startswith("build/js"):
             self.action = self.COMBOFILE
         else:
diff --git a/lib/lp/translations/browser/distribution.py b/lib/lp/translations/browser/distribution.py
index 6488042..8cba72b 100644
--- a/lib/lp/translations/browser/distribution.py
+++ b/lib/lp/translations/browser/distribution.py
@@ -88,7 +88,7 @@ class DistributionView(LaunchpadView):
     def translation_focus(self):
         """Return the IDistroSeries where the translators should work.
 
-        If ther isn't a defined focus, we return latest series.
+        If there isn't a defined focus, we return latest series.
         """
         if self.context.translation_focus is None:
             return self.context.currentseries
diff --git a/lib/lp/translations/browser/distributionsourcepackage.py b/lib/lp/translations/browser/distributionsourcepackage.py
index 3b6893c..6ec7c98 100644
--- a/lib/lp/translations/browser/distributionsourcepackage.py
+++ b/lib/lp/translations/browser/distributionsourcepackage.py
@@ -21,7 +21,7 @@ class DistributionSourcePackageView(LaunchpadView):
     def translation_focus(self):
         """Return the ISourcePackage where the translators should work.
 
-        If ther isn't a defined focus, we return latest series.
+        If there isn't a defined focus, we return latest series.
         """
         series = (
             self.context.distribution.translation_focus
diff --git a/lib/lp/translations/browser/language.py b/lib/lp/translations/browser/language.py
index 7a0b15a..1b7e5b0 100644
--- a/lib/lp/translations/browser/language.py
+++ b/lib/lp/translations/browser/language.py
@@ -262,7 +262,7 @@ class LanguageView(TranslationsMixin, LaunchpadView):
             else:
                 translator_target = translator
 
-            # Add translator only if it was not previouly added as a
+            # Add translator only if it was not previously added as a
             # merged account
             if translator_target not in top_translators:
                 top_translators.append(translator_target)
diff --git a/lib/lp/translations/browser/person.py b/lib/lp/translations/browser/person.py
index f7a5037..64fd9fc 100644
--- a/lib/lp/translations/browser/person.py
+++ b/lib/lp/translations/browser/person.py
@@ -346,7 +346,7 @@ class PersonTranslationView(LaunchpadView):
     ):
         """Add `new_targets` to `existing_targets` list.
 
-        This is for use in showing top-10 ists of translations a user
+        This is for use in showing top-10 lists of translations a user
         should help review or complete.
 
         :param existing_targets: Translation targets that are already
diff --git a/lib/lp/translations/browser/tests/test_pofile_view.py b/lib/lp/translations/browser/tests/test_pofile_view.py
index f68f2ff..4b0a90f 100644
--- a/lib/lp/translations/browser/tests/test_pofile_view.py
+++ b/lib/lp/translations/browser/tests/test_pofile_view.py
@@ -87,7 +87,7 @@ class TestQueryCount(TestCaseWithFactory):
 class TestPOFileTranslateViewInvalidFiltering(TestCaseWithFactory):
     """Test how POFile views reacts to malformed GET requests.
 
-    Since any number of parameters can be entered throug the URL, the view
+    Since any number of parameters can be entered through the URL, the view
     should be robust about them and not produce OOPSes. This is achieved by
     raising UnexpectedFormData which is communicated to the user instead of
     being recorded as an OOPS.
diff --git a/lib/lp/translations/browser/tests/test_sharing_details.py b/lib/lp/translations/browser/tests/test_sharing_details.py
index 4828730..3c648e1 100644
--- a/lib/lp/translations/browser/tests/test_sharing_details.py
+++ b/lib/lp/translations/browser/tests/test_sharing_details.py
@@ -254,7 +254,7 @@ class TestSourcePackageTranslationSharingDetailsView(
 
     def test_is_configuration_complete__packaging_upstream_branch_set(self):
         # If the packaging link is set and if an upstream branch is
-        # configuerd but if the other conditions are not fulfilled,
+        # configured but if the other conditions are not fulfilled,
         # is_configuration_complete is False.
         self.configureSharing(set_upstream_branch=True)
         self.assertFalse(self.view.is_configuration_complete)
diff --git a/lib/lp/translations/browser/tests/test_sharing_information.py b/lib/lp/translations/browser/tests/test_sharing_information.py
index bfaed87..1668a6f 100644
--- a/lib/lp/translations/browser/tests/test_sharing_information.py
+++ b/lib/lp/translations/browser/tests/test_sharing_information.py
@@ -62,7 +62,7 @@ class TestSharingInfoMixin:
     def getAuthorizedUserForProductseries(self, productseries):
         """Get a user that has Edit rights on productseries.
 
-        If productseries is None, return an arbritrary user. Used by
+        If productseries is None, return an arbitrary user. Used by
         implementations of getAuthorizedUser.
         """
         logged_in_user = self.factory.makePerson()
diff --git a/lib/lp/translations/browser/translationtemplatesbuild.py b/lib/lp/translations/browser/translationtemplatesbuild.py
index 60e8fd7..e1a3a4d 100644
--- a/lib/lp/translations/browser/translationtemplatesbuild.py
+++ b/lib/lp/translations/browser/translationtemplatesbuild.py
@@ -21,7 +21,7 @@ class TranslationTemplatesBuildView(LaunchpadView):
     """View for `TranslationTemplatesBuild`."""
 
     def getTargets(self):
-        """`ProducSeries` that will consume the generated templates."""
+        """`ProductSeries` that will consume the generated templates."""
         utility = getUtility(IProductSeriesSet)
         return list(
             utility.findByTranslationsImportBranch(self.context.branch)
@@ -35,7 +35,7 @@ class TranslationTemplatesBuildView(LaunchpadView):
             formatter.approximatedate(),
         )
 
-    def initalize(self):
+    def initialize(self):
         """See `LaunchpadView`."""
         self.last_score = HARDCODED_TRANSLATIONTEMPLATESBUILD_SCORE
 
diff --git a/lib/lp/translations/interfaces/pofiletranslator.py b/lib/lp/translations/interfaces/pofiletranslator.py
index 64331c8..1dd314e 100644
--- a/lib/lp/translations/interfaces/pofiletranslator.py
+++ b/lib/lp/translations/interfaces/pofiletranslator.py
@@ -65,7 +65,7 @@ class IPOFileTranslatorSet(Interface):
         """
 
     def getForTemplate(potemplate):
-        """Retrieve `POFileTranslator` objects associated iwth `POTemplate`.
+        """Retrieve `POFileTranslator` objects associated with `POTemplate`.
 
         :param potemplate: `POTemplate` to look for.
         :return: Result set of `POFileTranslator` records associated with
diff --git a/lib/lp/translations/interfaces/potemplate.py b/lib/lp/translations/interfaces/potemplate.py
index d394b64..92fcc0d 100644
--- a/lib/lp/translations/interfaces/potemplate.py
+++ b/lib/lp/translations/interfaces/potemplate.py
@@ -419,7 +419,7 @@ class IPOTemplate(IRosettaStats):
         """A key for determining the sharing precedence of a template.
 
         Active templates have precedence over inactive ones.
-        Development foci have precendence over non-development foci.
+        Development foci have precedence over non-development foci.
         Product development foci have precedence over Package development
         foci.
         """
diff --git a/lib/lp/translations/interfaces/potmsgset.py b/lib/lp/translations/interfaces/potmsgset.py
index 30aa50d..b3cdbd8 100644
--- a/lib/lp/translations/interfaces/potmsgset.py
+++ b/lib/lp/translations/interfaces/potmsgset.py
@@ -215,7 +215,7 @@ class IPOTMsgSet(Interface):
         self.getExternallyUsedTranslationMessages for each language.
 
         :param suggested_languages: languages we want suggestions for.
-        :param used_languages: languges we want used messages for.
+        :param used_languages: languages we want used messages for.
         """
 
     def hasTranslationChangedInLaunchpad(potemplate, language):
diff --git a/lib/lp/translations/interfaces/translationimportqueue.py b/lib/lp/translations/interfaces/translationimportqueue.py
index 459760d..92e6c32 100644
--- a/lib/lp/translations/interfaces/translationimportqueue.py
+++ b/lib/lp/translations/interfaces/translationimportqueue.py
@@ -247,7 +247,7 @@ class ITranslationImportQueueEntry(Interface):
             "The IPOTemplate associated with this entry. If path"
             " notes a .pot file, it should be used as the place where this"
             " entry will be imported, if it's a .po file, it indicates the"
-            " template associated with tha translation."
+            " template associated with the translation."
         ),
         required=False,
     )
diff --git a/lib/lp/translations/interfaces/translationmessage.py b/lib/lp/translations/interfaces/translationmessage.py
index b8e68ee..c2ab498 100644
--- a/lib/lp/translations/interfaces/translationmessage.py
+++ b/lib/lp/translations/interfaces/translationmessage.py
@@ -41,7 +41,7 @@ class RosettaTranslationOrigin(DBEnumeratedType):
         Source Control Management Source
 
         This translation sighting came from a PO File we
-        analysed in a source control managements sytem first.
+        analysed in a source control managements system first.
         """,
     )
 
diff --git a/lib/lp/translations/model/pofile.py b/lib/lp/translations/model/pofile.py
index 06a70ce..be5ec5e 100644
--- a/lib/lp/translations/model/pofile.py
+++ b/lib/lp/translations/model/pofile.py
@@ -1805,7 +1805,7 @@ class POFileToTranslationFileDataAdapter:
         header_nplurals = header.number_plural_forms
         database_nplurals = self._pofile.language.pluralforms
         # These checks are here to catch cases where the plural information
-        # from the header might be more acurate than what we have in the
+        # from the header might be more accurate than what we have in the
         # database. This is usually the case when the number of plural forms
         # has grown but not if it's the standard western form.
         # See bug 565294
diff --git a/lib/lp/translations/model/translationimportqueue.py b/lib/lp/translations/model/translationimportqueue.py
index 0b7cc8c..4c9184f 100644
--- a/lib/lp/translations/model/translationimportqueue.py
+++ b/lib/lp/translations/model/translationimportqueue.py
@@ -112,7 +112,7 @@ def compose_approval_conflict_notice(domain, templates_count, sample):
     """Create a note to warn about an approval conflict.
 
     The note warns about the situation where one productseries, or source
-    package, or in some cases distroseries has multiple actice templates
+    package, or in some cases distroseries has multiple active templates
     with the same translation domain.
 
     :param domain: The domain that's causing trouble.
diff --git a/lib/lp/translations/model/translationsharingjob.py b/lib/lp/translations/model/translationsharingjob.py
index 0031914..950143b 100644
--- a/lib/lp/translations/model/translationsharingjob.py
+++ b/lib/lp/translations/model/translationsharingjob.py
@@ -33,9 +33,9 @@ class TranslationSharingJobType(DBEnumeratedType):
     PACKAGING_MERGE = DBItem(
         0,
         """
-        Merge translations betweeen productseries and sourcepackage.
+        Merge translations between productseries and sourcepackage.
 
-        Merge translations betweeen productseries and sourcepackage.
+        Merge translations between productseries and sourcepackage.
         """,
     )
 
diff --git a/lib/lp/translations/tests/test_pofile.py b/lib/lp/translations/tests/test_pofile.py
index e9f73c4..a6ed46d 100644
--- a/lib/lp/translations/tests/test_pofile.py
+++ b/lib/lp/translations/tests/test_pofile.py
@@ -1750,7 +1750,7 @@ class TestPOFileSet(TestCaseWithFactory):
         pofile1.date_changed = week_ago
 
         # Let's make sure the condition from the bug holds,
-        # since pofile2 is created implicitely with the makePOTemplate call.
+        # since pofile2 is created implicitly with the makePOTemplate call.
         self.assertTrue(pofile1.id < pofile2.id)
         pofiles = self.pofileset.getPOFilesTouchedSince(yesterday)
         self.assertContentEqual([pofile1, pofile2], pofiles)
@@ -2577,7 +2577,7 @@ class TestPOFile(TestCaseWithFactory):
         )
 
     def test_prepare_pomessage_error_message_sequence_is_invalid(self):
-        # The errordetails can be contructed when the sequnce is invalid.
+        # The errordetails can be constructed when the sequence is invalid.
         errors = [
             {
                 "potmsgset": self.factory.makePOTMsgSet(
diff --git a/lib/lp/translations/tests/test_potemplate.py b/lib/lp/translations/tests/test_potemplate.py
index f4fd40c..72f2fd8 100644
--- a/lib/lp/translations/tests/test_potemplate.py
+++ b/lib/lp/translations/tests/test_potemplate.py
@@ -576,7 +576,7 @@ class TestTemplatePrecedence(TestCaseWithFactory):
 
 
 class TestTranslationFoci(TestCaseWithFactory):
-    """Test the precedence rules for tranlation foci."""
+    """Test the precedence rules for translation foci."""
 
     layer = DatabaseFunctionalLayer
 
diff --git a/lib/lp/translations/tests/test_potmsgset.py b/lib/lp/translations/tests/test_potmsgset.py
index 6e180af..640a3c2 100644
--- a/lib/lp/translations/tests/test_potmsgset.py
+++ b/lib/lp/translations/tests/test_potmsgset.py
@@ -852,7 +852,7 @@ class TestPOTMsgSetSuggestions(TestCaseWithFactory):
             ),
         )
         # Dismiss suggestions using an older timestamp fails if there is
-        # a newer curent translation.
+        # a newer current translation.
         self.assertRaises(
             TranslationConflict,
             self.potmsgset.dismissAllSuggestions,
diff --git a/lib/lp/translations/tests/test_productserieslanguage.py b/lib/lp/translations/tests/test_productserieslanguage.py
index 9295811..64468ee 100644
--- a/lib/lp/translations/tests/test_productserieslanguage.py
+++ b/lib/lp/translations/tests/test_productserieslanguage.py
@@ -182,7 +182,7 @@ class TestProductSeriesLanguageStatsCalculation(TestCaseWithFactory):
         )
         self.assertEqual(psl.messageCount(), 0)
 
-        # We explicitely ask for stats to be recalculated.
+        # We explicitly ask for stats to be recalculated.
         psl.recalculateCounts()
 
         self.assertPSLStatistics(
@@ -213,7 +213,7 @@ class TestProductSeriesLanguageStatsCalculation(TestCaseWithFactory):
         self.setPOFileStatistics(pofile2, 1, 1, 1, 1, pofile2.date_changed)
 
         psl = self.productseries.productserieslanguages[0]
-        # We explicitely ask for stats to be recalculated.
+        # We explicitly ask for stats to be recalculated.
         psl.recalculateCounts()
 
         # Total is a sum of totals in both POTemplates (10+20).
diff --git a/lib/lp/translations/tests/test_setcurrenttranslation.py b/lib/lp/translations/tests/test_setcurrenttranslation.py
index 0962aac..fd1b478 100644
--- a/lib/lp/translations/tests/test_setcurrenttranslation.py
+++ b/lib/lp/translations/tests/test_setcurrenttranslation.py
@@ -215,7 +215,7 @@ class SetCurrentTranslationTestMixin:
         self.assertTrue(tm is not None)
         if follows:
             # Even if the other side is supposed to follow this side,
-            # we ovverride the other only if the current side is Ubuntu.
+            # we override the other only if the current side is Ubuntu.
             expected_other = self.selectUpstreamTranslation(tm, tm_other)
         else:
             expected_other = tm_other
diff --git a/lib/lp/translations/tests/test_translationmessage.py b/lib/lp/translations/tests/test_translationmessage.py
index 00ed418..6ad2292 100644
--- a/lib/lp/translations/tests/test_translationmessage.py
+++ b/lib/lp/translations/tests/test_translationmessage.py
@@ -1114,7 +1114,7 @@ class TestShareIfPossible(TestCaseWithFactory):
         self.assertFalse(other_translation.is_current_ubuntu)
 
     def test_current_upstream_no_clash(self):
-        """If tranlation is current-upstream, and there's no clash, delete."""
+        """If translation is current-upstream, and there's no clash, delete."""
         translation, other_translation = self.makeUnsharedTranslation(
             is_current_upstream=True, clashing_upstream=False
         )
diff --git a/lib/lp/translations/tests/test_translations_to_review.py b/lib/lp/translations/tests/test_translations_to_review.py
index fbe8e07..bac5cb7 100644
--- a/lib/lp/translations/tests/test_translations_to_review.py
+++ b/lib/lp/translations/tests/test_translations_to_review.py
@@ -181,7 +181,7 @@ class TestReviewableProductTranslationFiles(
         ReviewTestMixin.setUpMixin(self, for_product=True)
 
     def test_getReviewableTranslationFiles_project_deactivated(self):
-        # Deactive project are excluded from the list.
+        # Deactivated projects are excluded from the list.
         from lp.testing import celebrity_logged_in
 
         with celebrity_logged_in("admin"):
diff --git a/lib/lp/translations/utilities/gettext_po_parser.py b/lib/lp/translations/utilities/gettext_po_parser.py
index f43b18d..e0332dd 100644
--- a/lib/lp/translations/utilities/gettext_po_parser.py
+++ b/lib/lp/translations/utilities/gettext_po_parser.py
@@ -195,7 +195,7 @@ class POHeader:
 
     def _decode(self, text):
         if text is None or isinstance(text, str):
-            # There is noo need to do anything.
+            # There is no need to do anything.
             return text
         charset = self.charset
         try:
diff --git a/lib/lp/translations/utilities/rosettastats.py b/lib/lp/translations/utilities/rosettastats.py
index 2f76757..ebb0ab5 100644
--- a/lib/lp/translations/utilities/rosettastats.py
+++ b/lib/lp/translations/utilities/rosettastats.py
@@ -31,25 +31,25 @@ class RosettaStats:
 
     def messageCount(self):
         """See IRosettaStats."""
-        # This method should be overrided by the objects that inherit from
+        # This method should be overridden by the objects that inherit from
         # this class
         return 0
 
     def currentCount(self, language=None):
         """See IRosettaStats."""
-        # This method should be overrided by the objects that inherit from
+        # This method should be overridden by the objects that inherit from
         # this class
         return 0
 
     def updatesCount(self, language=None):
         """See IRosettaStats."""
-        # This method should be overrided by the objects that inherit from
+        # This method should be overridden by the objects that inherit from
         # this class
         return 0
 
     def rosettaCount(self, language=None):
         """See IRosettaStats."""
-        # This method should be overrided by the objects that inherit from
+        # This method should be overridden by the objects that inherit from
         # this class
         return 0
 
diff --git a/lib/lp/translations/utilities/tests/helpers.py b/lib/lp/translations/utilities/tests/helpers.py
index f5a0e8f..487aa6a 100644
--- a/lib/lp/translations/utilities/tests/helpers.py
+++ b/lib/lp/translations/utilities/tests/helpers.py
@@ -30,7 +30,7 @@ def import_pofile_or_potemplate(
     :param pofile: if uploading a `POFile`, file to import to; None otherwise.
     :param potemplate: if uploading a `POTemplate`, file to import to; None
         otherwise.
-    :param by_maintainer: true if this file was uplaoded by the maintainer
+    :param by_maintainer: true if this file was uploaded by the maintainer
         of the project or package.
     :return: `TranslationImportQueueEntry` as added to the import queue.
     """
diff --git a/lib/lp/translations/utilities/tests/test_file_importer.py b/lib/lp/translations/utilities/tests/test_file_importer.py
index fa3a82e..b33596e 100644
--- a/lib/lp/translations/utilities/tests/test_file_importer.py
+++ b/lib/lp/translations/utilities/tests/test_file_importer.py
@@ -531,7 +531,7 @@ class FileImporterTestCase(TestCaseWithFactory):
     def test_FileImporter_importFile_error(self):
         # Test that a validation error is handled correctly during import.
         # This is done by trying to store a translation (msgstr) with format
-        # spefifiers that do not match those in the msgid, as they should.
+        # specifiers that do not match those in the msgid, as they should.
         (pot_importer, po_importer) = self._createFileImporters(
             TEST_TEMPLATE_FOR_ERROR, TEST_TRANSLATION_FILE_WITH_ERROR, False
         )
diff --git a/lib/lp/translations/utilities/translation_export.py b/lib/lp/translations/utilities/translation_export.py
index e2e3114..e09264f 100644
--- a/lib/lp/translations/utilities/translation_export.py
+++ b/lib/lp/translations/utilities/translation_export.py
@@ -189,7 +189,7 @@ class TarballFileStorageStrategy(StorageStrategy):
     empty = False
 
     def __init__(self, single_file_storage=None):
-        """Initialze empty storage strategy, or subsume single-file one."""
+        """Initialize empty storage strategy, or subsume single-file one."""
         self.buffer = tempfile.TemporaryFile()
         self.tar_writer = LaunchpadWriteTarFile(self.buffer)
         if single_file_storage is not None:
diff --git a/lib/lp/translations/vocabularies.py b/lib/lp/translations/vocabularies.py
index 501f89c..ff69730 100644
--- a/lib/lp/translations/vocabularies.py
+++ b/lib/lp/translations/vocabularies.py
@@ -181,7 +181,7 @@ class FilteredLanguagePackVocabulary(FilteredLanguagePackVocabularyBase):
     @property
     def _clauses(self):
         # We are interested on any full language pack or language pack
-        # that is a delta of the current base lanuage pack type,
+        # that is a delta of the current base language pack type,
         # except the ones already used.
         used_lang_packs = []
         if self.context.language_pack_base is not None:
diff --git a/lib/lp/xmlrpc/tests/test_private_xmlrpc.py b/lib/lp/xmlrpc/tests/test_private_xmlrpc.py
index 3b1e31a..66e8dea 100644
--- a/lib/lp/xmlrpc/tests/test_private_xmlrpc.py
+++ b/lib/lp/xmlrpc/tests/test_private_xmlrpc.py
@@ -70,7 +70,7 @@ class TestPrivateXMLRPC(TestCase):
             self.assertStartsWith(result, "http://bugs.launchpad.test/bugs/")
 
     def test_internal_bugs_api(self):
-        """There is an interal bugs api, too, but that doesn't share the same
+        """There is an internal bugs api, too, but that doesn't share the same
         methods as those exposed publicly.
         """
         internal_api = self.get_private_proxy("bugs/")