Propose removing "versioning" argument from loadpkl

[stilley2]

The "versioning" argument to loadpkl no longer serves much purpose. I propose it be deleted. Thoughts?

nipype/nipype/utils/filemanip.py

Line 678 in 508df5e

def loadpkl(infile, versioning=False):

[effigies]

cc @anibalsolon, as this was added in #2626.

[anibalsolon]

Hi @stilley2 ! I am actually not aware of why it isn't required anymore. But if it is not, I am always happy with cutting off code.

[stilley2]

Hi @anibalsolon , if my reading of the code is correct, the only thing it changes now is error reporting. loadpkl tries to read the version information regardless of if this flag is set, and if that fails it assumes it doesn't exist.

[anibalsolon]

Yes @stilley2 so it can be removed indeed!