Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

chore: update the Usage part of readme #33

Merged
merged 1 commit into from
Apr 22, 2021

Conversation

zeigo
Copy link
Contributor

@zeigo zeigo commented Apr 7, 2021

The Usage part of readme missed the parameter for simple.NewBlockstoreProvider()

@welcome
Copy link

welcome bot commented Apr 7, 2021

Thank you for submitting this PR!
A maintainer will be here shortly to review it.
We are super grateful, but we are also overloaded! Help us by making sure that:

  • The context for this PR is clear, with relevant discussion, decisions
    and stakeholders linked/mentioned.

  • Your contribution itself is clear (code comments, self-review for the
    rest) and in its best form. Follow the code contribution
    guidelines

    if they apply.

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.
Next steps:

  • A maintainer will triage and assign priority to this PR, commenting on
    any missing things and potentially assigning a reviewer for high
    priority items.

  • The PR gets reviews, discussed and approvals as needed.

  • The PR is merged by maintainers when it has been approved and comments addressed.

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.
We are very grateful for your contribution!

@Stebalien Stebalien merged commit f194251 into ipfs:master Apr 22, 2021
@zeigo zeigo deleted the chore/update-readme branch April 22, 2021 08:38
Jorropo pushed a commit to ipfs/go-libipfs-rapide that referenced this pull request Mar 23, 2023
…eadme

chore: update the Usage part of readme

This commit was moved from ipfs/go-ipfs-provider@f194251
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants