Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add codespell: config, workflow (to alert when new typos added) and get typos fixed #29

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add codespell: config, workflow (to alert when new typos added) and get typos fixed #29

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c31de43
Add github action to codespell master on push and PRs
yarikoptic Dec 6, 2023
6bcfc96
Add rudimentary codespell config
yarikoptic Dec 6, 2023
d46e45d
[DATALAD RUNCMD] run codespell throughout fixing typo automagically
yarikoptic Dec 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .codespellrc
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
[codespell]
skip = .git,*.pdf,*.svg,.codespellrc
check-hidden = true
# ignore-regex =
ignore-words-list = nd
22 changes: 22 additions & 0 deletions .github/workflows/codespell.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
---
name: Codespell

on:
push:
branches: [master]
pull_request:
branches: [master]

permissions:
contents: read

jobs:
codespell:
name: Check for spelling errors
runs-on: ubuntu-latest

steps:
- name: Checkout
uses: actions/checkout@v3
- name: Codespell
uses: codespell-project/actions-codespell@v2
2 changes: 1 addition & 1 deletion DAV-requirements.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,4 +49,4 @@ partial put.
There seems to be some movement in this space. RFC9110 mentions
[partial PUT using the `Content-Range` header](https://www.rfc-editor.org/rfc/rfc9110.html#name-partial-put), even though that is very unsafe - on servers not supporting it you'll get instant data corruption.

Seperately, there's a [Byte Range PATCH](https://datatracker.ietf.org/doc/draft-wright-http-patch-byterange/) draft RFC. This one looks better, let's hope it goes forward.
Separately, there's a [Byte Range PATCH](https://datatracker.ietf.org/doc/draft-wright-http-patch-byterange/) draft RFC. This one looks better, let's hope it goes forward.
2 changes: 1 addition & 1 deletion SABREDAV-partialupdate.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The `Sabre\\DAV\\PartialUpdate\\Plugin` from the Sabre DAV library provides
support for the HTTP PATCH method [RFC5789](http://tools.ietf.org/html/rfc5789).
This allows you to update just a portion of a file, or append to a file.

This document can be used as a spec for other implementors. There is some
This document can be used as a spec for other implementers. There is some
DAV-specific stuff in this document, but only in relation to the OPTIONS
request.

Expand Down
2 changes: 1 addition & 1 deletion webdav.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -378,7 +378,7 @@ func (d *DavClient) Mount() (err error) {
d.IsApache = true
}

// Does this server supoort sabredav-partialupdate ?
// Does this server support sabredav-partialupdate ?
if d.DavSupport["sabredav-partialupdate"] {
d.IsSabre = true
}
Expand Down