P. L. Lim

Results 2091 comments of P. L. Lim

Haven't noticed any problems. Thanks again! 🙇‍♀️

CI is green now. Are there any remaining unresolved comments?

Re: 5.2 or 6.0 -- Up to you. If you want it in 6.0, this can never be merged before 5.2.x branching, so safer to turn it back into draft....

`Read-only file system: '/.astropy'` is the main problem here. Please see https://docs.astropy.org/en/stable/config/index.html on how to change the default to point to a place you have write access to.

Quick comments without carefully reading the whole PR... @astrofrog tried something like this a while ago so maybe he can have a look. He had commented that this was hard...

Yes, remote data job does not have coverage. If you want to do due diligence, you can always run coverage locally with remote data turned on. 😸

I am not excited at another refactor of `utils` if we can help it. Lots of people use it, including HPC people.

> whether there is a minimum supported version number that we should fix for fsspec? Won't hurt to specify one.

I think this is a big enough feature that warrants a what's new entry for v5.2. Could you please write one? Thanks!