Make the maximum asset filename length configurable.

[D3X]

Some filesystems do not support filenames that are 255 characters long. Notably, aufs used by CircleCI only supports filenames up to 242 characters.
By making this a configurable option, we allow users to set their desired length limit.

[BeyondEvil]

Awesome!

Should this be an ini-option instead of a command line option? 🤔

[D3X]

@BeyondEvil I have changed it to be an ini option and force-pushed. It's failing linting, because flake8 has a different rule than black regarding whitespace before colons. Probably something to be fixed in a separate PR.

[BeyondEvil]

@BeyondEvil I have changed it to be an ini option and force-pushed. It's failing linting, because flake8 has a different rule than black regarding whitespace before colons. Probably something to be fixed in a separate PR.

Interesting, I'll pull your PR and check it out.

[BeyondEvil]

@D3X

Please just add ignore = E203 under the [flake8] section in tox.ini

[BeyondEvil]

Sorry @D3X I should've been more clear, apart from adding ignore = E203 you also have to put that space before colon. That's why linting failed now.

As a side note, are you using pre-commit with this project? It would've caught the above before committing. Instructions here

[D3X]

@BeyondEvil sorry it took so long, I was quite busy the last couple of weeks. The build's green now, so hopefully we can get this merged and released soon-ish ;)

[BeyondEvil]

LGTM!