Issue 32446: ResourceLoader.get_data() should accept a PathLike (original) (raw)

Created on 2017-12-29 15:33 by barry, last changed 2022-04-11 14:58 by admin. This issue is now closed.

Messages (6)
msg309178 - (view) Author: Barry A. Warsaw (barry) * (Python committer) Date: 2017-12-29 15:33
Currently get_data() only accepts a string. It should also accept a os.PathLike
msg310798 - (view) Author: Jay Yin (jayyin11043) * Date: 2018-01-26 21:25
I'm taking a look now, this looks interesting
msg310801 - (view) Author: Jay Yin (jayyin11043) * Date: 2018-01-26 21:31
this would extend to set and get data right?, also would it be good to use the current get_data() (one that reads only string) function for the overloaded one and extract the string path from the os.path object? or would we want the os.path object to be handled separately?
msg310802 - (view) Author: Jay Yin (jayyin11043) * Date: 2018-01-26 21:36
nvm I temporarily forgot I was working with python here, I'd need to use "is" to check what the entered "path" is huh?
msg310804 - (view) Author: Jay Yin (jayyin11043) * Date: 2018-01-26 21:41
I hope this isn't a dumb question but where is ResourceLoader.get_data() implemented, using ATOM to search the whole project and I can't seem to find it.
msg310808 - (view) Author: Barry A. Warsaw (barry) * (Python committer) Date: 2018-01-26 22:04
Hi Jay. Thanks for your interest in contributing to Python! Actually, I'm going to close this issue since we ultimately decided to deprecate the ResourceLoader ABC: https://docs.python.org/3.7/library/importlib.html?highlight=resourceloader#importlib.abc.ResourceLoader I'd also suggest that if you're interested in answering your questions for the curiosity, or want to contribute to Python in other areas, you should join the python-dev mailing list.
History
Date User Action Args
2022-04-11 14:58:56 admin set github: 76627
2018-01-26 22:04:33 barry set status: open -> closedresolution: wont fixmessages: + stage: resolved
2018-01-26 21:41:19 jayyin11043 set messages: +
2018-01-26 21:36:33 jayyin11043 set messages: +
2018-01-26 21:31:36 jayyin11043 set messages: +
2018-01-26 21:25:01 jayyin11043 set nosy: + jayyin11043messages: +
2017-12-29 15:33:19 barry create