Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: set default value of lint to false to improve the execution per… #7425

Merged
merged 5 commits into from
Mar 25, 2025

Conversation

JerryWu1234
Copy link
Contributor

@JerryWu1234 JerryWu1234 commented Mar 15, 2025

…formance

What is it?

fixes #6837
related to 190
set default value of lint to false to improve the execution performance'

Description

Checklist

  • My code follows the developer guidelines of this project
  • I performed a self-review of my own code
  • I added a changeset with pnpm change
  • I made corresponding changes to the Qwik docs
  • I added new tests to cover the fix / functionality

@JerryWu1234 JerryWu1234 requested a review from a team as a code owner March 15, 2025 02:05
Copy link

changeset-bot bot commented Mar 15, 2025

🦋 Changeset detected

Latest commit: cba3161

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

pkg-pr-new bot commented Mar 15, 2025

Open in Stackblitz

npm i https://pkg.pr.new/@builder.io/qwik@7425
npm i https://pkg.pr.new/@builder.io/qwik-city@7425
npm i https://pkg.pr.new/eslint-plugin-qwik@7425
npm i https://pkg.pr.new/create-qwik@7425

commit: cba3161

Copy link
Contributor

github-actions bot commented Mar 15, 2025

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
qwik-docs ✅ Ready (View Log) Visit Preview df076ec

@JerryWu1234 JerryWu1234 self-assigned this Mar 15, 2025
Copy link
Contributor

@maiieul maiieul left a comment

Choose a reason for hiding this comment

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

@JerryWu1234 Imo we should change that default in the optimizer's vite.ts or plugin.ts, not in the starters.

@JerryWu1234 JerryWu1234 marked this pull request as draft March 15, 2025 10:09
@JerryWu1234 JerryWu1234 marked this pull request as ready for review March 17, 2025 10:58
@JerryWu1234
Copy link
Contributor Author

@JerryWu1234 Imo we should change that default in the optimizer's vite.ts or plugin.ts, not in the starters.

it's fine right now

@shairez shairez dismissed maiieul’s stale review March 25, 2025 20:38

seems like it's fixed now

Copy link
Contributor

@shairez shairez left a comment

Choose a reason for hiding this comment

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

LGTM
Thanks @JerryWu1234 !

@shairez shairez enabled auto-merge (squash) March 25, 2025 20:39
@shairez shairez merged commit 66d8c7f into QwikDev:main Mar 25, 2025
17 checks passed
@github-actions github-actions bot mentioned this pull request Mar 25, 2025
@@ -0,0 +1,5 @@
---
'base': patch
Copy link
Contributor

Choose a reason for hiding this comment

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

just noticed this (I fixed it already in a different commit)

but for next time, please verify that this has a real package name or else it breaks the release

thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[🐞] qwikVite lint:true can slowdown development servers significantly
4 participants