vlajos / misspell-fixer Goto Github PK

I love the project, thanks a lot! I just ran it on one of my projects and saw some Permission denied errors for temporary, gitignored cache files.

As I don't really care for typos in files that aren't part of my repository, it'd be great to have a flag to just ignore them.

This mostly identical to #34. The current script implementations seems to skip such strings. But a typo can be hidden in the middle of the SomeBigCamelCaseStrng and nobody can ever notice that :)

"initilized" should be replaced by "initialized" but is not detected even if -V is set.

It seems misspell-fixer will fix spelling mistakes inside .jar files which are binary Java distribution files ( https://en.wikipedia.org/wiki/JAR_(file_format) ). I believe the extension should be included in the set of binary extensions.

We are running misspell-fixer as part of a Github Action. As such, I'd like to keep all configuration for these actions within our .github folder, and specify a path of .github/.misspell-fixer.ignore for the ignore file. Reading the documentation (and after a cursory look at the source code), it doesn't seem possible to specify a different location for this file. I'd love for there to be a command line argument to override the default path to this configuration file.

We have a few small video files used for testing, in particular these are .mov and .mp4 format. The misspell-fixer changed these files. I guess that runs a risk of corrupting the files though.

There is a known bug of sed -i on Windows MinGW that the file is set to read-only. The workaround is to call attrib -R afterwards.

Particularly, it removes the executable bit from files.

Please consider adding an option for exceptions like .min.css and .min.js.
Köszönöm!

This is a very nice project.

cp: target `space/foo.txt.30985' is not a directory

Maybe it should be (quotes added).

cp --attributes-only "$1" $tmpfile

Jó napot kívánok!

isspell_fixer: actual file: ../w3c/resource-hints//index.html
misspell_fixer: temp file: ../w3c/resource-hints//index.html.72361
cp: illegal option -- -
usage: cp [-R [-H | -L | -P]] [-fi | -n] [-apvX] source_file target_file
       cp [-R [-H | -L | -P]] [-fi | -n] [-apvX] source_file ... target_directory
misspell_fixer: nothing changed
misspell_fixer: actual file: ../w3c/resource-hints//README.md
misspell_fixer: temp file: ../w3c/resource-hints//README.md.72367
cp: illegal option -- -
usage: cp [-R [-H | -L | -P]] [-fi | -n] [-apvX] source_file target_file
       cp [-R [-H | -L | -P]] [-fi | -n] [-apvX] source_file ... target_directory
misspell_fixer: nothing changed
misspell_fixer: Done.

Blame old (default) bash, but.. still, would be nice if it worked :)

Right now script seems to miss checks for the strings like My_Sting_With_Underscore.

It would be a great addition (especially when you want to check the code base for misprints) to the current implementation if such strings could be detected and split and then concatenated back :)

Thanks!

I usually type locla.
Could we add locla → local?

I run misspell_fixer.sh on Win32 using either MinGW or Cygwin environment. It would be nice if CRLF line endings are preserved (and not changed to LF automatically). What I do now is to run unix2dos for every changed file afterwards.