Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

refactor: convert object.* and files.* API methods to async/await #1160

Merged
merged 8 commits into from
Nov 20, 2019

Conversation

alanshaw
Copy link
Contributor

@alanshaw alanshaw commented Nov 15, 2019

This also allows the timeout option to be passed as a human readable string, which is compatible with the go-ipfs global timeout option.

Depends on:

@alanshaw alanshaw changed the title refactor: convert some object and files API methods to async/await refactor: convert object.* and files.* API methods to async/await Nov 15, 2019
@alanshaw alanshaw force-pushed the refactor/object-and-files-async-await branch from 1ebd7c8 to b7b4a91 Compare November 18, 2019 15:58
@alanshaw alanshaw changed the title refactor: convert object.* and files.* API methods to async/await refactor: convert object.put/patch and files.* API methods to async/await Nov 18, 2019
@alanshaw alanshaw marked this pull request as ready for review November 18, 2019 17:03
@alanshaw alanshaw changed the title refactor: convert object.put/patch and files.* API methods to async/await refactor: convert object.* and files.* API methods to async/await Nov 20, 2019
@@ -27,7 +28,7 @@ module.exports = create => config => {
// https://github.com/sindresorhus/ky/pull/153
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we still need this ? its been merged

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not the PR I thought it was, it still gets fetch off the global object so we can't pass a custom fetch implementation. This comment should just be removed.

@alanshaw alanshaw merged commit fc73da7 into master Nov 20, 2019
@alanshaw alanshaw deleted the refactor/object-and-files-async-await branch November 20, 2019 17:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants