Skip to content

Remove dist from tracking and update readme#5

Merged
harryalbert merged 1 commit intomainfrom
harry/update-dist-and-ci
Apr 6, 2026
Merged

Remove dist from tracking and update readme#5
harryalbert merged 1 commit intomainfrom
harry/update-dist-and-ci

Conversation

@harryalbert
Copy link
Copy Markdown
Contributor

@harryalbert harryalbert commented Apr 6, 2026

We shouldn't be publishing the dist to this repo (it only matters for npm publish).

We also need to update the readme to use the correct plugin path for installation instructions.

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@harryalbert harryalbert marked this pull request as ready for review April 6, 2026 20:41
@harryalbert harryalbert requested a review from zachbai April 6, 2026 20:42
@harryalbert harryalbert force-pushed the harry/update-dist-and-ci branch from 9ad7fe3 to 10a9c60 Compare April 6, 2026 20:48
@harryalbert harryalbert changed the title update dist and add test Remove dist from tracking and update readme Apr 6, 2026
"test": "bun test",
"dev": "echo 'Add to your opencode.json: \"plugin\": [\"file://'$(pwd)'/src/index.ts\"]' && echo 'Then run: opencode'"
"dev": "echo 'Add to your opencode.json: \"plugin\": [\"file://'$(pwd)'/src/index.ts\"]' && echo 'Then run: opencode'",
"prepublishOnly": "tsc"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

that auto-compiles the dist before we publish it so that we don't get stale dists in the future

@harryalbert harryalbert merged commit 3018f3b into main Apr 6, 2026
2 checks passed
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