Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: test example building #122

Merged
merged 9 commits into from
Dec 20, 2024
Merged

ci: test example building #122

merged 9 commits into from
Dec 20, 2024

Conversation

ocavue
Copy link
Contributor

@ocavue ocavue commented Dec 20, 2024

Run the build script for all examples during CI pipeline. The build script for the examples/svelte has been broken for a while. This PR can prevent it from happening again.

Copy link

pkg-pr-new bot commented Dec 20, 2024

Open in Stackblitz

@prosemirror-adapter/core

npm i https://pkg.pr.new/prosekit/prosemirror-adapter/@prosemirror-adapter/core@122

@prosemirror-adapter/react

npm i https://pkg.pr.new/prosekit/prosemirror-adapter/@prosemirror-adapter/react@122

@prosemirror-adapter/lit

npm i https://pkg.pr.new/prosekit/prosemirror-adapter/@prosemirror-adapter/lit@122

@prosemirror-adapter/solid

npm i https://pkg.pr.new/prosekit/prosemirror-adapter/@prosemirror-adapter/solid@122

@prosemirror-adapter/vue

npm i https://pkg.pr.new/prosekit/prosemirror-adapter/@prosemirror-adapter/vue@122

@prosemirror-adapter/svelte

npm i https://pkg.pr.new/prosekit/prosemirror-adapter/@prosemirror-adapter/svelte@122

commit: c27f397

@ocavue ocavue force-pushed the ocavue/example-build branch from 671ce0a to 1563c4e Compare December 20, 2024 11:38
@ocavue ocavue changed the title ci: test example build ci: test example building Dec 20, 2024
@ocavue ocavue requested a review from Saul-Mirone December 20, 2024 11:50
@@ -0,0 +1 @@
link-workspace-packages=deep
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I need this so that pnpm -r run build can build packages first then build examples. Using "@prosemirror-adapter/lit": "workspace:*" in package.json can do this job too, but it won't allow stackblitz to open a sub directory like examples/lit and install dependencies.

@ocavue
Copy link
Contributor Author

ocavue commented Dec 20, 2024

@Saul-Mirone Could you review this? I removed .npmrc from .gitignore and I want to make you aware of it.

Copy link
Collaborator

@Saul-Mirone Saul-Mirone left a comment

Choose a reason for hiding this comment

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

LGTM 🚒

@ocavue ocavue merged commit 7049b72 into main Dec 20, 2024
4 checks passed
@ocavue ocavue deleted the ocavue/example-build branch December 20, 2024 16:10
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