Skip to content

ci : add build of wasm examples to CI #2959

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

Closed
wants to merge 4 commits into from

Conversation

danbev
Copy link
Member

@danbev danbev commented Mar 28, 2025

This commit add the building of the wasm examples to CI.

Refs: #2784

danbev added 3 commits March 28, 2025 10:45
This commit add the building of the wasm examples to CI.

Refs: ggml-org#2784
This commit adds src and include to the paths filter in the examples
workflow.

The motivation for this is that to avoid missing changes in src/include
that may effect the examples we should also run them when src/include
change.
@danbev
Copy link
Member Author

danbev commented Mar 28, 2025

CI run for wasm examples. I'll revert the dummy commit shortly.

@danbev
Copy link
Member Author

danbev commented Mar 28, 2025

Thinking about this a little more perhaps there should be an examples-wasm.yml, examples-node.yml etc. The downside of what I've done here when adding to the existing examples.yml is that the addon.node example will also run when changes to the .wasm examples are made and vice versa, which is not great. Alternatively there are actions that can be used to provide more fine grained filtering per job like paths-changes-filter.

@ggerganov
Copy link
Member

We can improve the workflows if needed. The main requirement is that the CI is fast. It's OK if it sometimes triggers builds that are not related to the changes - as long as the triggered workflows are quick, then it's fine.

Copy link
Member

@ggerganov ggerganov left a comment

Choose a reason for hiding this comment

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

Very nice addition. Should we try to create an action that generates Github Pages, hosting the examples? I'm not sure how difficult that is, though.

@danbev
Copy link
Member Author

danbev commented Mar 30, 2025

We can improve the workflows if needed. The main requirement is that the CI is fast.

I just realized that this is actually already covered by the emscripten job. I'll close this so that we don't duplicated this job.

Should we try to create an action that generates Github Pages, hosting the examples?

I think that would be very nice, I'll take a look 👍

@danbev danbev closed this Mar 30, 2025
@danbev danbev deleted the ci-wasm-examples-build branch April 1, 2025 03:54
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