Skip to content

Formalize vaporview api in npm package#168

Open
AndrewNolte wants to merge 2 commits intoLramseyer:mainfrom
AndrewNolte:AndrewNolte/stack/10
Open

Formalize vaporview api in npm package#168
AndrewNolte wants to merge 2 commits intoLramseyer:mainfrom
AndrewNolte:AndrewNolte/stack/10

Conversation

@AndrewNolte
Copy link
Copy Markdown
Contributor

@AndrewNolte AndrewNolte commented Mar 27, 2026

@AndrewNolte AndrewNolte force-pushed the AndrewNolte/stack/10 branch from 876632e to 81eb639 Compare March 27, 2026 18:38
@AndrewNolte AndrewNolte marked this pull request as draft March 27, 2026 18:46
@AndrewNolte AndrewNolte marked this pull request as ready for review March 27, 2026 18:47
@AndrewNolte AndrewNolte marked this pull request as draft March 27, 2026 18:48
@AndrewNolte AndrewNolte force-pushed the AndrewNolte/stack/10 branch from 81eb639 to 83576cc Compare March 27, 2026 18:48
@AndrewNolte AndrewNolte marked this pull request as ready for review March 27, 2026 18:48
@AndrewNolte AndrewNolte marked this pull request as draft March 27, 2026 18:57
@AndrewNolte AndrewNolte force-pushed the AndrewNolte/stack/10 branch from 83576cc to 9f6265e Compare March 27, 2026 18:57
@AndrewNolte AndrewNolte marked this pull request as ready for review March 27, 2026 18:57
@AndrewNolte AndrewNolte marked this pull request as draft March 27, 2026 20:19
@AndrewNolte AndrewNolte force-pushed the AndrewNolte/stack/10 branch from 9f6265e to 5011e18 Compare March 27, 2026 20:19
@AndrewNolte AndrewNolte marked this pull request as ready for review March 27, 2026 20:19
@AndrewNolte AndrewNolte marked this pull request as draft March 27, 2026 21:50
@AndrewNolte AndrewNolte force-pushed the AndrewNolte/stack/10 branch from 5011e18 to aa19ff5 Compare March 27, 2026 21:50
Lramseyer and others added 2 commits March 27, 2026 17:50
stack-info: PR: Lramseyer#168, branch: AndrewNolte/stack/10
return {
const api: VaporviewApi = {
// Events
onDidSetMarker: markerSetEvent,
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Doesn't this type need to be changed?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants