Skip to content

Comments

Add smoke test and CI yaml#141

Merged
jkwak-work merged 2 commits intoshader-slang:mainfrom
jkwak-work:add-smoke-test-and-ci
May 3, 2025
Merged

Add smoke test and CI yaml#141
jkwak-work merged 2 commits intoshader-slang:mainfrom
jkwak-work:add-smoke-test-and-ci

Conversation

@jkwak-work
Copy link
Contributor

@jkwak-work jkwak-work commented Apr 30, 2025

Adding CI yaml.
Adding smoke-test.js
Improve the build scripts

@jkwak-work
Copy link
Contributor Author

Closes shader-slang/slang#6957

@jkwak-work jkwak-work requested a review from kaizhangNV April 30, 2025 10:22
@jkwak-work
Copy link
Contributor Author

I am making this as a draft.
I am going to move the smoke-test.js to slang repo somewhere and use it on both slang and slang-playground.

@jkwak-work jkwak-work marked this pull request as draft April 30, 2025 19:13
@jkwak-work jkwak-work force-pushed the add-smoke-test-and-ci branch from 1026bfc to 3764b28 Compare May 1, 2025 19:26
@jkwak-work jkwak-work marked this pull request as ready for review May 1, 2025 19:27
@jkwak-work
Copy link
Contributor Author

smoke-test.js is merged in slang repo.
This PR will use smoke-test.js from slang repo, because the repo has to be cloned anyway.

Copy link
Collaborator

@kaizhangNV kaizhangNV left a comment

Choose a reason for hiding this comment

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

LGTM

@jkwak-work jkwak-work merged commit 154831b into shader-slang:main May 3, 2025
2 checks passed
@jkwak-work jkwak-work deleted the add-smoke-test-and-ci branch May 3, 2025 04:35
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