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: production smoke test #1353

Merged
merged 25 commits into from
Mar 3, 2024
Merged

ci: production smoke test #1353

merged 25 commits into from
Mar 3, 2024

Conversation

KermanX
Copy link
Member

@KermanX KermanX commented Feb 28, 2024

Not very sure about the name of this CI.

This PR adds a CI that simulates the user's behavior.

Conditions (2x3x2 + 1x2x2 = 16):

  • NPM / PNPM / YARN
  • Use a package and install Slidev locally / Use a single md file and install Slidev globally
  • Hoist or non-hoist when installed locally
  • Ubuntu or Windows

Note that MacOS is not included in this test, because it always OOM when building:
image

Also, YARN global install hasn't been included yet. It seems that YARN's global install is not very common.

Also, the following error happens randomly at a low frequency, but fixing this is out of this PR:
image

Visit https://github.com/KermanX/slidev-add-ci/actions to see the output of this CI in a copy of Slidev.

@antfu
Copy link
Member

antfu commented Feb 28, 2024

Great! I think we could also try cypress e2e test against production build?

@antfu antfu changed the title ci: add user-side test ci: production smoke test Mar 3, 2024
@antfu antfu merged commit c3f1380 into slidevjs:main Mar 3, 2024
20 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