Code review comment for ~therealchjones/casper:supported-fs

Revision history for this message
Christian Jones (therealchjones) wrote :

Yes, I think that’s what I’ve done. The only fs in the function at line 35 that’s not in the function at line 361 is udf. I added that to the list in the function at line 361, then removed the whole function at 35. (As it’s defined first, that’s the one that hasn’t been running, of course.) unless I’ve made some very silly mistake in the merge request (which is always possible), the result should be a minimally modified version of the function that’s otherwise been working properly and removal of the function that’s been overridden since the previous commit. Did I screw it up somehow?

« Back to merge proposal