See https://flit.readthedocs.io/en/latest/pyproject_toml.html#build-system-section

This will allow flakehell installable from just the source. E.g. pip install git+git+https://github.com/life4/flakehell

From my testing, passing --config ./file.toml wasn't being handled correctly. Debugging it, it looks like parse_preliminary_options in flake8 strips out these options, so by the time we get to it in parse_configuration_and_cli they are nowhere to be found in argv.

• Adding the pylint symbol (verbal error code) makes disabling certain errors a bit easier.
• Printing error codes in only one color makes them a bit more readable and from my experience, there is no real need to make the number stand out.

The pylint issue is more important for me. If you have any objections against the other change, I can revert it. :)

In CI environment, depending on the user and whether it has a home, flakehell would try to create the /.cache directory, and would fail with a PermissionError.

This commit changes that by making flakehell try to create the cache at different locations: $HOME/.cache/flakehell and ./.flakehell_cache (the chances we can write into the current directory in CI are high).

A --disable-cache option would be much more robust but seems to be way more complicated to implement.

The exclude option is still broken in 0.4.5, I still get a SyntaxError on a file that is in the exclude list.

I'm also struggling to figure out how this would be used as a git hook.

The error is caused by Snapshot class naively reading in file content to create a md5 digest for the file. This is not possible when - is passed, because a) it is most likely not an existing filename, and b) even if we were to interpret it as stdin, it isn't seekable (outside of using os.lseek when a real file was redirected).

The fix: when calculating digests for a non-existent file, just invent one from current time and a random value. It will be a new value every time, ensuring no caching between such stdin invocations.

Fixes #44

Currently flakehell is using 2 versions:

• flakehell.__version__ which is pypi package version
• flakehell._constants.VERSION which is passed to Flake8Application as version kwarg

Maybe it's some leftover after refactor or something, but it's quite confusing and imho only one version number should be used for both cases. Please correct me if I am wrong, because maybe I misunderstood something.

Thanks for maintaining this really useful project!

• It is easily confusable with my project (I've already received reports that are your bugs and not mine)
• It is a derivative work and violation of my copyright without attribution -- the code in your project directly mirrors the structure of mine and copies substantial parts including the regex patterns and processing code

Not sure if the current behavior is by design or not. When some file is matched by two or more exceptions, only one exception is applied and it is not obvious which one (the one with a shorter path actually). So, i suggest to apply all matched exceptions.

A PermissionError can be raised when trying to create the cache directory. We allow the user to change the cache path with the FLAKEHELL_CACHE environment variable to work around this issue.

Replaces #101.

This will allow baseline files to be generated on any platform to be consumed on any platform.

NOTE: Unfortunately, this will break existing baseline files generated on machines that use back slashes.

From my testing, passing --config ./file.toml wasn't being handled correctly. Debugging it, it looks like parse_preliminary_options in flake8 strips out these options, so by the time we get to it in parse_configuration_and_cli they are nowhere to be found in argv.