Skip to content

Fix spelling errors in the help files#553

Open
dforsi wants to merge 1 commit intook2cqr:masterfrom
dforsi:fix/typos
Open

Fix spelling errors in the help files#553
dforsi wants to merge 1 commit intook2cqr:masterfrom
dforsi:fix/typos

Conversation

@dforsi
Copy link
Copy Markdown

@dforsi dforsi commented Jan 14, 2023

This PR fixes some small errors in the help files.
I didn't replace the lower case "qsl", "qso", "qth" with upper case because it would double the size of this diff for little gain.

I reverted an automatic fix by codespell in help/sql_console.html where it would replace "releas" with "release" because that's the actual field name in src/dData.lfm

CREATE TABLE version (
            major INTEGER DEFAULT 0,
            minor INTEGER DEFAULT 9,
            releas INTEGER DEFAULT 4
);

codespell could be run also on the source files making it to ignore some words like "releas" but there is still the risk of breaking the code. For example the following command lists all the word that would be changed if you run it add a --write-changes option (and --interactive=2 to let you choose when there is more than one choice):
codespell --ignore-words-list=ans,dout,release,ro,ser,te --skip=./ctyfiles/*,./members/*,./zipcodes/* --summary
however there are probably more words to ignore and directories to skip.

Fixed with:
codespell --write-changes help/
codespell --write-changes help/ --interactive=2
ispell help/*.html
@AsciiWolf
Copy link
Copy Markdown
Contributor

LGTM!

This should be merged after the merge conflict gets resolved.

/cc @ok2cqr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants