Merge ~cjwatson/launchpad:codespell-2.2.5 into launchpad:master

Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: e5bbed389a8d4381092041533a6ad1350afc98a2
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cjwatson/launchpad:codespell-2.2.5
Merge into: launchpad:master
Diff against target: 167 lines (+17/-16)
10 files modified
.codespell-ignore (+1/-0)
.pre-commit-config.yaml (+1/-1)
lib/lp/answers/tests/emailinterface.rst (+2/-2)
lib/lp/app/javascript/tests/test_lp_client.js (+1/-1)
lib/lp/blueprints/stories/blueprints/xx-superseding-within-projects.rst (+2/-2)
lib/lp/registry/javascript/tests/test_structural_subscription.js (+3/-3)
lib/lp/registry/stories/announcements/xx-announcements.rst (+1/-1)
lib/lp/services/doc/sprites.rst (+1/-1)
lib/lp/soyuz/doc/archive.rst (+4/-4)
lib/lp/soyuz/stories/webservice/xx-archive.rst (+1/-1)
Reviewer Review Type Date Requested Status
Guruprasad Approve
Review via email: mp+445884@code.launchpad.net

Commit message

Upgrade to codespell 2.2.5

Description of the change

This involved a number of spelling corrections, and ignoring "optiona" which legitimately turns up in `lib/lp/registry/stories/team-polls/edit-options.rst` (as "OptionA") but looks like a misspelling of "optional".

To post a comment you must log in.
Revision history for this message
Guruprasad (lgp171188) wrote :

LGTM 👍

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/.codespell-ignore b/.codespell-ignore
2index 1d1422b..7af0ae1 100644
3--- a/.codespell-ignore
4+++ b/.codespell-ignore
5@@ -7,6 +7,7 @@ fpr
6 importd
7 iterm
8 lightening
9+optiona
10 serieses
11 sting
12 tread
13diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
14index dab74e2..ead3e42 100644
15--- a/.pre-commit-config.yaml
16+++ b/.pre-commit-config.yaml
17@@ -82,7 +82,7 @@ repos:
18 hooks:
19 - id: shellcheck
20 - repo: https://github.com/codespell-project/codespell
21- rev: v2.2.2
22+ rev: v2.2.5
23 hooks:
24 - id: codespell
25 args: ["-I", ".codespell-ignore"]
26diff --git a/lib/lp/answers/tests/emailinterface.rst b/lib/lp/answers/tests/emailinterface.rst
27index fa182bc..d8fa1f3 100644
28--- a/lib/lp/answers/tests/emailinterface.rst
29+++ b/lib/lp/answers/tests/emailinterface.rst
30@@ -242,8 +242,8 @@ From the Expired state:
31
32 From the Expired state, the other possibility is the less probable
33 explaining that the owner solved their problem. Again, to minimize
34-confusion, the outoing notification contain a footer explaining what
35-will happen if one reply to the message.
36+confusion, the outgoing notification contain a footer explaining what
37+will happen if one replies to the message.
38
39 Solved and Invalid
40 ..................
41diff --git a/lib/lp/app/javascript/tests/test_lp_client.js b/lib/lp/app/javascript/tests/test_lp_client.js
42index 5c00536..b99cd13 100644
43--- a/lib/lp/app/javascript/tests/test_lp_client.js
44+++ b/lib/lp/app/javascript/tests/test_lp_client.js
45@@ -519,7 +519,7 @@ YUI.add('lp.client.test', function (Y) {
46 null, this.response, this.args);
47
48 // If the response does not include a notifications header, then
49- // any pre-existing notifiactions are not removed.
50+ // any pre-existing notifications are not removed.
51 this.response.setResponseHeader('X-Lazr-Notifications', null);
52 Y.lp.client.wrap_resource_on_success(
53 null, this.response, this.args);
54diff --git a/lib/lp/blueprints/stories/blueprints/xx-superseding-within-projects.rst b/lib/lp/blueprints/stories/blueprints/xx-superseding-within-projects.rst
55index 365caa3..ed2e705 100644
56--- a/lib/lp/blueprints/stories/blueprints/xx-superseding-within-projects.rst
57+++ b/lib/lp/blueprints/stories/blueprints/xx-superseding-within-projects.rst
58@@ -1,5 +1,5 @@
59-Superseeding blueprints
60-=======================
61+Superseding blueprints
62+======================
63
64 Bueprints can only be superseded by other blueprints in the same project.
65 The user interface code for the superseding form passes blueprints by their
66diff --git a/lib/lp/registry/javascript/tests/test_structural_subscription.js b/lib/lp/registry/javascript/tests/test_structural_subscription.js
67index 2bd6853..dbec3e9 100644
68--- a/lib/lp/registry/javascript/tests/test_structural_subscription.js
69+++ b/lib/lp/registry/javascript/tests/test_structural_subscription.js
70@@ -1054,7 +1054,7 @@ YUI.add('lp.structural_subscription.test', function (Y) {
71 }));
72
73 tests.suite.add(new Y.Test.Case({
74- name: 'Structural Subscription: add subcription workflow',
75+ name: 'Structural Subscription: add subscription workflow',
76
77 setUp: function() {
78 var TestBugFilter = function() {};
79@@ -1176,7 +1176,7 @@ YUI.add('lp.structural_subscription.test', function (Y) {
80
81 test_description_is_added: function() {
82 // If we add a subscription on a page that doesn't display
83- // subcription details then we need to add an "informational
84+ // subscription details then we need to add an "informational
85 // message" describing the just-added subscription.
86 var handler;
87 var config = {add_filter_description: false};
88@@ -1190,7 +1190,7 @@ YUI.add('lp.structural_subscription.test', function (Y) {
89 }));
90
91 tests.suite.add(new Y.Test.Case({
92- name: 'Structural Subscription: edit subcription workflow',
93+ name: 'Structural Subscription: edit subscription workflow',
94
95 setUp: function() {
96 var TestBugFilter = function(data) {
97diff --git a/lib/lp/registry/stories/announcements/xx-announcements.rst b/lib/lp/registry/stories/announcements/xx-announcements.rst
98index 9161479..a051a28 100644
99--- a/lib/lp/registry/stories/announcements/xx-announcements.rst
100+++ b/lib/lp/registry/stories/announcements/xx-announcements.rst
101@@ -411,7 +411,7 @@ any links to show announcements.
102 >>> count_show_links(anon_browser.contents)
103 0
104
105-Distribution pages may have the link in the annoucements portlet,
106+Distribution pages may have the link in the announcements portlet.
107
108 >>> anon_browser.open("http://launchpad.test/ubuntu")
109 >>> count_show_links(anon_browser.contents)
110diff --git a/lib/lp/services/doc/sprites.rst b/lib/lp/services/doc/sprites.rst
111index a582bd1..c6c0270 100644
112--- a/lib/lp/services/doc/sprites.rst
113+++ b/lib/lp/services/doc/sprites.rst
114@@ -41,7 +41,7 @@ A SpriteUtil object currently can only process a single group.
115 Loading the CSS Template
116 ------------------------
117
118-Instatiating a new SpriteUtil object will parse the css template and
119+Instantiating a new SpriteUtil object will parse the css template and
120 find all the css rules for the specified group. The url_prefix_substitutions
121 parameter allows you to convert url prefixes into file paths relative
122 to the directory passed into combineImages(). The space between sprites
123diff --git a/lib/lp/soyuz/doc/archive.rst b/lib/lp/soyuz/doc/archive.rst
124index 3c14ec2..00b89b5 100644
125--- a/lib/lp/soyuz/doc/archive.rst
126+++ b/lib/lp/soyuz/doc/archive.rst
127@@ -1019,15 +1019,15 @@ package copy request:
128 >>> from lp.testing.factory import (
129 ... remove_security_proxy_and_shout_at_engineer,
130 ... )
131- >>> requestor = factory.makePerson(name="me-copy")
132+ >>> requester = factory.makePerson(name="me-copy")
133 >>> copy_target = factory.makeCopyArchiveLocation(
134- ... distribution=ubuntu, name="my-copy-archive", owner=requestor
135+ ... distribution=ubuntu, name="my-copy-archive", owner=requester
136 ... )
137 >>> naked_copy_target = remove_security_proxy_and_shout_at_engineer(
138 ... copy_target
139 ... )
140 >>> pcr = ubuntu.main_archive.requestPackageCopy(
141- ... naked_copy_target, requestor
142+ ... naked_copy_target, requester
143 ... )
144 >>> print(pcr)
145 Package copy request
146@@ -1041,7 +1041,7 @@ package copy request:
147 The requestPackageCopy method can also take an optional suite name:
148
149 >>> package_copy_request = ubuntu.main_archive.requestPackageCopy(
150- ... naked_copy_target, requestor, suite="hoary-updates"
151+ ... naked_copy_target, requester, suite="hoary-updates"
152 ... )
153 >>> print(package_copy_request)
154 Package copy request
155diff --git a/lib/lp/soyuz/stories/webservice/xx-archive.rst b/lib/lp/soyuz/stories/webservice/xx-archive.rst
156index a2a2242..b53d7d3 100644
157--- a/lib/lp/soyuz/stories/webservice/xx-archive.rst
158+++ b/lib/lp/soyuz/stories/webservice/xx-archive.rst
159@@ -1484,7 +1484,7 @@ be able to access Celso's private PPA.
160 ... )
161
162 Note that the 'description' and the 'signing_key_fingerprint'
163-attributes are only exposed when the requestor has View permission in
164+attributes are only exposed when the requester has View permission in
165 the IArchive context, in this case only Celso has it.
166
167 >>> pprint_entry(

Subscribers

People subscribed via source and target branches

to status/vote changes: