-
Notifications
You must be signed in to change notification settings - Fork 197
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Usage instructions out-of-date #818
Comments
This is a good catch. Would you be up for submitting a PR? |
I don't know about "out of date", none of those sound compatible with any On Wednesday, 15 June 2016, Tom Hunt notifications@github.com wrote:
|
I could make a PR with examples of the invocations that have worked for me. I don't know enough about the underlying concepts to really correct the descriptions. (For instance, what exactly is a 'beak'? Docs say a cell |
Hm. Fair. To be clear, you're referring to: http://urbit.org/docs/using/filesystem/ ? I think @ohAitch will handle this. |
On it. A beak is definitely neither a ship-desk cell nor On Wednesday, 15 June 2016, Galen Wolfe-Pauly notifications@github.com
|
@ohAitch should we merge your branch? |
Yup, when I made the PR I thought docs also had "master == deploy" semantics |
Well, it looks like the PR didn't get closed for a few minor issues. You're free to close it when it's ready, of course. And yeah: master gets deployed periodically. |
Ah right, the |
Various usage instructions and examples under
using/filesystem
are now apparently out of date. For instance:|merge %desk [~planet-name %desk]
now givesnest-fail
, and only functions with|merge %desk /~planet-name/desk=
;|sync %desk ~planet-name
fails, requiring instead|sync %desk ~planet-name %desk
(docs state the remote desk is optional if the same as the local desk);|cp /path1 /path2 %init
fails, requiring no merge strategy argument.The text was updated successfully, but these errors were encountered: