Skip to content

fix various typos #237

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.

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

Open
wants to merge 24 commits into
base: hovudstraum
Choose a base branch
from
Open

fix various typos #237

wants to merge 24 commits into from

Conversation

luzpaz
Copy link

@luzpaz luzpaz commented Jul 28, 2025

Found via codespell -q 3 -L ascript,caf,clen,fo,nd,oen,pres,som,te,tread,warks -S "*.patch"
This PR complies with the DCO; https://developercertificate.org/

@@ -11,7 +11,7 @@ var hambagas = [
"https://www.youtube.com/watch?v=pFA3KGp4GuU"
];

// keybaord,
// keyboard,
// onscreen keyboard by @steinuil
function initKeybaord(BASE_URL, HAMBAGA, consoleLog, consoleError) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

initKeybaord should be renamed to initKeyboard.

Copy link
Author

@luzpaz luzpaz Jul 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you prefer for me to make source changes in this PR or a separate one ?

Edit:
Screenshot_20250730_142258

@@ -373,7 +373,7 @@ function initKeybaord(BASE_URL, HAMBAGA, consoleLog, consoleError) {
}


// keybaord integration
// keyboard integration
(function () {
var o = mknod('div');
clmod(o, 'keybaord-container', 1);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

keybaord-container?

@toast003
Copy link
Contributor

Found a typo while reviewing the Spanish translation PR

"mm_eunk": "Unknown Errol",

9001 and others added 23 commits July 30, 2025 18:30
The version attribute is deprecated, resolves error "the attribute `version` is obsolete, it will be ignored, please remove it to avoid potential confusion" when building with Docker

Signed-off-by: ptweezy <[email protected]>
Comic readers will sort alphabetically, but that isn't always the order in which the files are stored in the zip.
* add feature showcase video

Signed-off-by: Adam <[email protected]>

* add youtube link too

Signed-off-by: ed <[email protected]>

---------

Signed-off-by: Adam <[email protected]>
Signed-off-by: ed <[email protected]>
Co-authored-by: ed <[email protected]>
* nix: allow passing extra packages in PATH

* nix: allow passing extra python packages

I wanted to use
https://github.com/9001/copyparty/blob/hovudstraum/bin/hooks/notify.py
but that wasn't really possible without this under the nix package.

* nix: format all nix files with nixfmt

* nix: reduce redundancy in the package

For readability

* nix: remove unused pyftpdlib import

* nix: put makeWrapper into the correct inputs

* nix: fill out all of meta

* nix: set formatter in flake for nix files

This allows contributors to format their nix changes with the `nix fmt`
command.

* nix: add u2c

* nix: add partyfuse

One downside of the way the nix ecosystem works is that MacFUSE needs to
be installed manually. Luckily the script tells you that already!

* nix: add missing cfssl import

* nix: add flake check that makes sure it builds with all flags

Because sometimes an import might be missing, and if it is an optional
then you'll only figure out that it's broken if you set the flag.

* nix: use correct overlay argument names

Or `nix flake check` will refuse to run the copyparty-full check
almost zero performance impact with podman in kitty
also makes rejections from IdP auths less confusing;
it was handled by the config-parser throwing "invalid config"
Found via `codespell -q 3 -L ascript,caf,clen,fo,nd,oen,pres,som,te,tread,warks -S "*.patch"`
This PR complies with the DCO; https://developercertificate.org/
@luzpaz
Copy link
Author

luzpaz commented Jul 30, 2025

Crap. Thought i rebased it on the correct branch ?

The relevant commits are the last 2 HEAD~2

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.

9 participants