Skip to content

Commit 120e693

Browse files
committed
[DATALAD RUNCMD] run codespell throughout fixing typos automagically
=== Do not change lines below === { "chain": [], "cmd": "codespell -w", "exit": 0, "extra_inputs": [], "inputs": [], "outputs": [], "pwd": "." } ^^^ Do not change lines above ^^^
1 parent d26e4bf commit 120e693

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

55 files changed

+90
-90
lines changed

.github/ISSUE_TEMPLATE/feature.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
name: Feature/improvement proposal
3-
about: Propose a new feauture or improvement for Read the Docs
3+
about: Propose a new feature or improvement for Read the Docs
44
---
55

66
## What's the problem this feature will solve?

.github/workflows/pip-tools.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# Action to run pip-compile weekly and create a Pull Request with the changes.
2-
# Althought GitHub says that pip-compile is supported by dependabot, we couldn't make it work together.
2+
# Although GitHub says that pip-compile is supported by dependabot, we couldn't make it work together.
33
# That's why this action exists.
44
# If we ever find the proper configuration for dependabot+pip-compile,
55
# we can delete this action.

CHANGELOG.rst

Lines changed: 13 additions & 13 deletions
Large diffs are not rendered by default.

docs/Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
# You can set these variables from the command line.
55

66
# Options should be aligned with how the documentation is built on RTD
7-
# (except for '--keep-going', failing fast is prefered on local builds)
7+
# (except for '--keep-going', failing fast is preferred on local builds)
88
# We also remove '-E' to reduce the time of rebuilding reference indexes
99
# on each build.
1010
SPHINXOPTS = -T -j auto -W

docs/dev/design/better-doc-urls-handling.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ Instead of trying to map a URL to a view,
4242
we first analyze the root project (given from the subdomain),
4343
and based on that we map each part of the URL to the *current* project and version.
4444

45-
This will allow us to re-use this code in our unresolver
45+
This will allow us to reuse this code in our unresolver
4646
without the need to override the Django's urlconf at runtime,
4747
or guessing a project only by the structure of its URL.
4848

docs/dev/design/build-images.rst

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -265,7 +265,7 @@ and we can roll back if the new pre-compiled version was built with a problem.
265265

266266
Installing always the latest version is harder to maintain.
267267
It will require building the newest version each time a new patch version is released.
268-
Beacause of that, Read the Docs will always be behind official releases.
268+
Because of that, Read the Docs will always be behind official releases.
269269
Besides, it will give projects different versions more often.
270270

271271
Exposing to the user the patch version would require to cache many different versions ourselves,
@@ -424,10 +424,10 @@ that doesn't seem to be useful to have the same OS version with different states
424424
Allowing users to install extra languages by using the Config File will cover most of the support requests we have had in the past.
425425
It also will allow us to know more about how our users are using the platform to make future decisions based on this data.
426426
Exposing users how we want them to use our platform will allow us to be able to maintain it longer,
427-
than giving the option to select a specific Docker image by name that we can't guarrantee it will be frozen.
427+
than giving the option to select a specific Docker image by name that we can't guarantee it will be frozen.
428428

429429
Finally, having the ability to deprecate and *remove* pre-built images from our builders over time,
430-
will reduce the maintainance work required from the the core team.
430+
will reduce the maintenance work required from the the core team.
431431
We can always support all the languages versions by installing them at build time.
432432
The only required pre-built image for this are the OS ``-base`` images.
433433
In fact, even after decided to deprecate and removed a pre-built image from the builders,

docs/dev/design/new-notifications-system.rst

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ Goals
3434
* Ability to add non-error notifications from the build process
3535
* Add extra metadata associated to the notification: icon, header, body, etc
3636
* Support different types of notifications (e.g. error, warning, note, tip)
37-
* Re-use the new notification system for product updates (e.g. new features, deprecated config keys)
37+
* Reuse the new notification system for product updates (e.g. new features, deprecated config keys)
3838
* Message content lives on Python classes that can be translated and formatted with objects (e.g. Build, Project)
3939
* Message could have richer content (e.g. HTML code) to generate links and emphasis
4040
* Notifications have trackable state (e.g. unread (default)=never shown, read=shown, dismissed=don't show again, cancelled=auto-removed after user action)
@@ -43,7 +43,7 @@ Goals
4343
* Notifications can be attached to Project, Organization, Build and User models
4444
* Specific notifications can be shown under the user's bell icon
4545
* Easy way to cleanup notification on status changes (e.g. subscription failure notification is auto-deleted after CC updated)
46-
* Notifications attached to Organization/Project dissappear for all the users once they are dismissed by anyone
46+
* Notifications attached to Organization/Project disappear for all the users once they are dismissed by anyone
4747

4848

4949
Non-goals
@@ -88,7 +88,7 @@ This section shows all the classes and models involved for the notification syst
8888
.. note:: Accessing the database from the build process
8989

9090
Builders doesn't have access to the database due to security reasons.
91-
We had solved this limitation by creating an API endpoint the builder hits once they need to interact with the databse to get a ``Project``, ``Version`` and ``Build`` resources, create a ``BuildCommand`` resource, etc.
91+
We had solved this limitation by creating an API endpoint the builder hits once they need to interact with the database to get a ``Project``, ``Version`` and ``Build`` resources, create a ``BuildCommand`` resource, etc.
9292

9393
Besides, the build process is capable to trigger Celery tasks that are useful for managing more complex logic
9494
that also require accessing from and writing to the database.
@@ -116,7 +116,7 @@ and some helper logic to return in the API response.
116116
body = str
117117
icon = str
118118
icon_style = str(SOLID, DUOTONE)
119-
type = str(ERROR, WARINIG, NOTE, TIP)
119+
type = str(ERROR, WARNING, NOTE, TIP)
120120
121121
def get_display_icon(self):
122122
if self.icon:
@@ -201,7 +201,7 @@ It contains an identifier (``message_id``) pointing to one of the messages defin
201201
message_id = models.CharField(max_length=128)
202202
203203
# UNREAD: the notification was not shown to the user
204-
# READ: the notifiation was shown
204+
# READ: the notification was shown
205205
# DISMISSED: the notification was shown and the user dismissed it
206206
# CANCELLED: removed automatically because the user has done the action required (e.g. paid the subscription)
207207
state = models.CharField(
@@ -210,7 +210,7 @@ It contains an identifier (``message_id``) pointing to one of the messages defin
210210
db_index=True,
211211
)
212212
213-
# Makes the notification imposible to dismiss (useful for Build notifications)
213+
# Makes the notification impossible to dismiss (useful for Build notifications)
214214
dismissable = models.BooleanField(default=False)
215215
216216
# Show the notification under the bell icon for the user

docs/dev/design/new-search-api.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ like the projects, versions, and the query that were used in the final search.
139139
And the ``version``, ``project``, and ``project_alias`` attributes will
140140
now be objects.
141141

142-
We could just re-use the old response too,
142+
We could just reuse the old response too,
143143
since the only breaking changes would be the attributes now being objects,
144144
and we aren't adding any new information to those objects (yet).
145145
But also, re-using the current serializers shouldn't be a problem either.
@@ -230,7 +230,7 @@ the readthedocs.org/readthedocs.com domains.
230230
We have two types:
231231

232232
Project scoped search:
233-
Search files and versions of the curent project only.
233+
Search files and versions of the current project only.
234234

235235
Global search:
236236
Search files and versions of all projects in .org,

docs/dev/design/pr-builder.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ We might consider adding a ``VERSION_TYPES`` to the ``Version`` model.
4343
- If we go with ``VERSION_TYPES`` we can add something like ``pull_request`` alongside Tag and Branch.
4444

4545
We should add ``Version`` and ``Build`` Model Managers for PR and Regular Versions and Builds.
46-
The proposed names for PR and Regular Version and Build Mangers are ``external`` and ``internal``.
46+
The proposed names for PR and Regular Version and Build Managers are ``external`` and ``internal``.
4747

4848
We can then use ``Version.internal.all()`` to get all regular versions,
4949
``Version.external.all()`` to get all PR versions.

docs/dev/design/secure-api-access-from-builders.rst

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ Why attach tokens to users?
9898
---------------------------
9999

100100
Attaching tokens to users will ease the implementation,
101-
since we can re-use the code from knox package.
101+
since we can reuse the code from knox package.
102102

103103
Attaching tokens to projects only is possible,
104104
but it will require to manage the authentication manually.
@@ -108,7 +108,7 @@ An alternative is to use the DRF API key package, which doesn't require a user,
108108
but then if we wanted to extend this functionality to our normal APIs, we will have
109109
to implement the authentication manually.
110110

111-
Kepping backwards compatibility
111+
Keeping backwards compatibility
112112
-------------------------------
113113

114114
Access to write API V2 is restricted to superusers,
@@ -161,7 +161,7 @@ since it also handles authentication).
161161
Decision
162162
--------
163163

164-
Due to the fact that the required featues from knox are not released yet,
164+
Due to the fact that the required features from knox are not released yet,
165165
we have decided to use DRF API key instead.
166166

167167
Future work

0 commit comments

Comments
 (0)