[FIX] test-requirements: pin max supported version for fsspec deps

[simahawk]

Tries to fix #602

There were 3 modules w/ 3 different versions of fsspec + no cap on test deps.

Collateral fix: remove obsolete setup folder.

[OCA-git-bot]

Hi @lmignon,
some modules you are maintaining are being modified, check this out!

[simahawk]

@lmignon can you pls review the versions for fsspec?

[lmignon]

@lmignon can you pls review the versions for fsspec?

Hi Simone,

Thank you for the PR. Having different versions of the same library should not be an issue. Python will install the higher version, so there is no need to align them. Have you encountered any problems?

[simahawk]

Ciao! :)
That's not the problem it's just that is confusing to have 3 versions in requirements.txt.
I've done that while solving the problem of the other dependencies. See #602

[simahawk]

@lmignon gentle ping for a review :)

[lmignon]

@simahawk In my view, it’s not a good idea to try to standardise the minimum required version of fsspecs across all modules. This involves specific API changes required by each module independently of the others, but which are not required by all. Could you please remove the commit that aligns the fsspecs version? Once that’s done, I’ll be able to merge the changes.

[simahawk]

I understand your pt and I can remove the commit. However, in this CI you never test w/ the version pinned for the specific module.
And if you use 2 modules in your installation, you'll have to choose one.

[simahawk]

I moved that part to #607

[ArnauCForgeFlow]

LGTM!

[lmignon]

/ocabot merge nobump

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-603-by-lmignon-bump-nobump, awaiting test results.

[lmignon]

thank you @simahawk !

[OCA-git-bot]

Congratulations, your PR was merged at 49d69d5. Thanks a lot for contributing to OCA. ❤️