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

Add Qwik example #359

Merged
merged 15 commits into from
Sep 27, 2024
Merged

Add Qwik example #359

merged 15 commits into from
Sep 27, 2024

Conversation

Mivr
Copy link
Contributor

@Mivr Mivr commented Sep 20, 2024

Added a simple Qwik example as to: #347

The app and lib are both plain generation from the examples provided by Qwik.
The app is connected to the lib with a link in the monorepo and uses the library components.

I have not added tests yet, from what I have seen they are just using vite test for the front end tests, I suppose a UI test is needed? I am asking as the tests will be the same as a generic Vitest on the build side at least.

The dev cycle is slow at the moment or not working OK, due to: vitejs/vite#8619

There is a start command that works, but is slow (>5s per update).
Also there is a fast (<1s) command "devserver" that works well only for 1 library update.
For this to be resolved it seems some work on Vite is needed.


Changes are visible to end-users: no

Test plan

  • Covered by existing test cases

Copy link

aspect-workflows bot commented Sep 20, 2024

Test

2 test targets passed

Targets
//qwik/app:vitest [k8-fastbuild] 4s
//qwik/lib:vitest [k8-fastbuild] 3s

Total test execution time was 8s. 23 tests (92.0%) were fully cached saving 5s.


Format      Lint

qwik/lib/package.json Outdated Show resolved Hide resolved
@alexeagle
Copy link
Member

FYI @shairez in case you missed Bazel 😉

@Mivr Mivr changed the title [DRAFT]: Add Qwik example Add Qwik example Sep 25, 2024
qwik/app/package.json Outdated Show resolved Hide resolved
alt="Qwik Logo"
width={400}
height={147}
src="https://cdn.builder.io/api/v1/image/assets%2FYJIGb4i01jvw0SRdL5Bt%2F667ab6c2283d4c4d878fb9083aacc10f"
Copy link

Choose a reason for hiding this comment

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

seems like the (very) old logo of Qwik.
I guess this starter has been generated from an old version of Qwik?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the logo in the library template at the moment: https://github.com/QwikDev/qwik/blob/main/starters/apps/library/src/components/logo/logo.tsx

I can replace it with what you are using on the docs or something else if you prefer? (the docs are not using an SVG exactly, but I think I can make it work). Also do you need this change in the Qwik repo as well?
The docs logo I could find: https://github.com/QwikDev/qwik/blob/main/packages/docs/src/components/svgs/qwik-logo.tsx

Copy link

@shairez shairez Sep 29, 2024

Choose a reason for hiding this comment

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

@Mivr yeah good point, we need to update it, we'll do thanks!

@shairez
Copy link

shairez commented Sep 26, 2024

Great job guys!
Added a few comments

Copy link
Member

@alexeagle alexeagle left a comment

Choose a reason for hiding this comment

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

Thanks @Mivr !!

@alexeagle alexeagle merged commit 7617be1 into aspect-build:main Sep 27, 2024
5 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.

3 participants