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

chore: ts-ignore vite 5 types issue #5341

Merged
merged 1 commit into from
Oct 22, 2023
Merged

chore: ts-ignore vite 5 types issue #5341

merged 1 commit into from
Oct 22, 2023

Conversation

bluwy
Copy link
Contributor

@bluwy bluwy commented Oct 22, 2023

Overview

What is it?

  • Feature / enhancement
  • Bug
  • Docs / tests / types / typos

Description

Currently Qwik in vite-ecosystem-ci is failing in the build step as Vite 5 has removed the ssr.format option. This PR fixes it by adding @ts-ignore so it works in both Vite 4 and Vite 5.

Use cases and why

There are a few more PRs coming to Vite 5, and being able to make Qwik green allows us to anticipate potential breaking changes better.

Checklist:

  • My code follows the developer guidelines of this project
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • Added new tests to cover the fix / functionality

@netlify
Copy link

netlify bot commented Oct 22, 2023

👷 Deploy request for qwik-insights pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 325e2ba

Copy link
Member

@gioboa gioboa left a comment

Choose a reason for hiding this comment

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

Thanks for your help

@gioboa gioboa merged commit 374f25e into QwikDev:main Oct 22, 2023
20 checks passed
@bluwy bluwy deleted the ts-ignore-vite-5-types branch October 22, 2023 12:01
kodiakhq bot pushed a commit to ascorbic/unpic-img that referenced this pull request Oct 29, 2023
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [@builder.io/qwik](https://qwik.builder.io/) ([source](https://togithub.com/BuilderIO/qwik)) | [`1.2.14` -> `1.2.15`](https://renovatebot.com/diffs/npm/@builder.io%2fqwik/1.2.14/1.2.15) | [![age](https://developer.mend.io/api/mc/badges/age/npm/@builder.io%2fqwik/1.2.15?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@builder.io%2fqwik/1.2.15?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@builder.io%2fqwik/1.2.14/1.2.15?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@builder.io%2fqwik/1.2.14/1.2.15?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>BuilderIO/qwik (@&#8203;builder.io/qwik)</summary>

### [`v1.2.15`](https://togithub.com/BuilderIO/qwik/releases/tag/v1.2.15)

[Compare Source](https://togithub.com/BuilderIO/qwik/compare/v1.2.14...v1.2.15)

#### What's Changed

-   fix(vercel): \[vercel-edge adapter] allow to use the `node` target by [@&#8203;Schniz](https://togithub.com/Schniz) in [QwikDev/qwik#4361
-   refactor(optimizer): remove using resolvePackageData API from Vite by [@&#8203;bluwy](https://togithub.com/bluwy) in [QwikDev/qwik#5312
-   feat: add outDir support to qwikInsights by [@&#8203;intellix](https://togithub.com/intellix) in [QwikDev/qwik#5335
-   fix(cli): set args on interactive cli by [@&#8203;mpeg](https://togithub.com/mpeg) in [QwikDev/qwik#5334
-   chore: ts-ignore vite 5 types issue by [@&#8203;bluwy](https://togithub.com/bluwy) in [QwikDev/qwik#5341
-   chore(docs): remove warnings by [@&#8203;gioboa](https://togithub.com/gioboa) in [QwikDev/qwik#5345
-   style(eslint): convey a stricter restriction from `just` to `only` (examples) by [@&#8203;uceumice](https://togithub.com/uceumice) in [QwikDev/qwik#5340
-   feat(vite): allow disabling dev SSR server in vite by [@&#8203;mhevery](https://togithub.com/mhevery) in [QwikDev/qwik#5347
-   chore(docs): Small fix of file to edit by [@&#8203;nelsonprsousa](https://togithub.com/nelsonprsousa) in [QwikDev/qwik#5348
-   docs: Update index.mdx by [@&#8203;runarj](https://togithub.com/runarj) in [QwikDev/qwik#5351
-   fix: action redirect accidentally stripped searchparams by [@&#8203;mhevery](https://togithub.com/mhevery) in [QwikDev/qwik#5349
-   fix: remove cf pages stream polyfill by [@&#8203;mpeg](https://togithub.com/mpeg) in [QwikDev/qwik#5352
-   chore: updated twitter logo to X by [@&#8203;Kaushik080](https://togithub.com/Kaushik080) in [QwikDev/qwik#5357
-   docs: update React cheat sheet title by [@&#8203;mhevery](https://togithub.com/mhevery) in [QwikDev/qwik#5358
-   chore: 1.2.15 by [@&#8203;mhevery](https://togithub.com/mhevery) in [QwikDev/qwik#5359

#### New Contributors

-   [@&#8203;Schniz](https://togithub.com/Schniz) made their first contribution in [QwikDev/qwik#4361
-   [@&#8203;bluwy](https://togithub.com/bluwy) made their first contribution in [QwikDev/qwik#5312
-   [@&#8203;intellix](https://togithub.com/intellix) made their first contribution in [QwikDev/qwik#5335
-   [@&#8203;nelsonprsousa](https://togithub.com/nelsonprsousa) made their first contribution in [QwikDev/qwik#5348
-   [@&#8203;Kaushik080](https://togithub.com/Kaushik080) made their first contribution in [QwikDev/qwik#5357

**Full Changelog**: QwikDev/qwik@v1.2.14...v1.2.15

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 9pm on sunday" (UTC), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/ascorbic/unpic-img).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMS41IiwidXBkYXRlZEluVmVyIjoiMzcuMzEuNSIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->
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.

None yet

2 participants