search

solid / solid-spec

Solid specification draft 0.7.0
Creative Commons Zero v1.0 Universal
1.13k stars 103 forks source link

Clarify container deletion, fix #172 #177

Closed michielbdejong closed 5 years ago

michielbdejong commented 5 years ago

@fabiancook can we merge this, making the current spec like rmdir, and then you can open an issue for your proposal to make it like rm -rf?

fabiancook commented 5 years ago

This seems like it defeats the purpose of the discussion we're having on the issue. By leaving this off, there is no spec change for rm -rf, it's a clarification, where this is a restriction?

All but one implementation referenced would be not up to spec given this PR

michielbdejong commented 5 years ago

OK so then we disagree not only on what the desired situation is but also on what the current situation is. :) Let's discuss that on #172.

michielbdejong commented 5 years ago

@RubenVerborgh @acoburn I think this should address both your comments, can you re-review?

michielbdejong commented 5 years ago

great, thanks!