Skip to content
This repository was archived by the owner on Nov 6, 2018. It is now read-only.

WIP: generate API docs #62

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

WIP: generate API docs #62

wants to merge 2 commits into from

Conversation

sqs
Copy link
Member

@sqs sqs commented Sep 14, 2018

fix #46

I want this only to emit sourcegraph.d.ts API docs, not internal API docs, but I haven't figured out how to do that yet.

sqs added 2 commits September 13, 2018 18:19
@sqs
feat: remove sourcegraphURL from extension API
d2d7168 
It was not used and is not necessary.
@sqs
WIP: typedoc
2f831aa 
I want this only to emit sourcegraph.d.ts API docs, not internal API docs, but I haven't figured out how to do that yet.
@cspotcode
Copy link

Can you use @internal annotations plus tsc's stripInternal option?

@sqs
Copy link
Member Author

sqs commented Oct 12, 2018

I will try that. Thanks!

@chrismwendt
Copy link

Here's how to move this PR to https://github.com/sourcegraph/sourcegraph/tree/master/packages/sourcegraph-extension-api

cd browser-extensions
git format-patch master --stdout > /tmp/patch

cd ../sourcegraph
cat /tmp/patch | git am -3 --directory=packages/sourcegraph-extension-api/
# and fixup merge conflicts

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@chrismwendt chrismwendt chrismwendt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sqs @cspotcode @chrismwendt