-
Notifications
You must be signed in to change notification settings - Fork 1.2k
fix(rmlink): fix rmlink to match docs #4073
Conversation
Thank you for submitting this PR!
Getting other community members to do a review would be great help too on complex PRs (you can ask in the chats/forums). If you are unsure about something, just leave us a comment.
We currently aim to provide initial feedback/triaging within two business days. Please keep an eye on any labelling actions, as these will indicate priorities and status of your contribution. |
Resolves #3815
The test provided for
rmlink
has the object structured as:https://github.com/ipfs/js-ipfs/blob/master/packages/interface-ipfs-core/src/object/patch/rm-link.js#L49-L55
While the documentation has it defined as:
Method has been updated to match the variable names in the documentation and the example has been altered to match the example in the test case.