Skip to content

coreapi: extract interface#5978

Merged
Stebalien merged 4 commits intomasterfrom
feat/extract-api-iface
Feb 11, 2019
Merged

coreapi: extract interface#5978
Stebalien merged 4 commits intomasterfrom
feat/extract-api-iface

Conversation

@magik6k
Copy link
Copy Markdown
Member

@magik6k magik6k commented Feb 9, 2019

License: MIT
Signed-off-by: Łukasz Magiera <magik6k@gmail.com>
License: MIT
Signed-off-by: Łukasz Magiera <magik6k@gmail.com>
License: MIT
Signed-off-by: Łukasz Magiera <magik6k@gmail.com>
@magik6k magik6k added the topic/core-api Topic core-api label Feb 9, 2019
@magik6k magik6k requested a review from Kubuxu as a code owner February 9, 2019 00:42
@ghost ghost assigned magik6k Feb 9, 2019
@ghost ghost added the status/in-progress In progress label Feb 9, 2019
@magik6k magik6k mentioned this pull request Feb 9, 2019
51 tasks
License: MIT
Signed-off-by: Łukasz Magiera <magik6k@gmail.com>
@magik6k magik6k changed the title [WIP] coreapi: extract interface coreapi: extract interface Feb 9, 2019
@Kubuxu
Copy link
Copy Markdown
Member

Kubuxu commented Feb 9, 2019

That will be annoyingly long import name, we didn't think about it while deciding on naming scheme.

@Stebalien
Copy link
Copy Markdown
Member

My only comment is that the repo name doesn't match the go naming convention (would otherwise be go-ipfs-interface-core). But don't really mind either way.

@magik6k
Copy link
Copy Markdown
Member Author

magik6k commented Feb 10, 2019

The naming is to match ipfs-inactive/interface-js-ipfs-core#423 (comment)

@Stebalien
Copy link
Copy Markdown
Member

I'm not going to bikeshed this too much but I've pinged david to try to get his rational. I do like being able to run shell commands over go-*.

@Stebalien Stebalien merged commit f6f9cf3 into master Feb 11, 2019
@ghost ghost removed the status/in-progress In progress label Feb 11, 2019
@Stebalien
Copy link
Copy Markdown
Member

We can quibble over naming later.

@Stebalien Stebalien deleted the feat/extract-api-iface branch February 11, 2019 16:33
hacdias pushed a commit to ipfs/boxo that referenced this pull request Jan 27, 2023
coreapi: extract interface

This commit was moved from ipfs/kubo@f6f9cf3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic/core-api Topic core-api

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants