Skip to content

fix: simplify dev mode check#3498

Merged
mrlubos merged 1 commit intomainfrom
fix/dev-mode-check
Mar 4, 2026
Merged

fix: simplify dev mode check#3498
mrlubos merged 1 commit intomainfrom
fix/dev-mode-check

Conversation

@mrlubos
Copy link
Member

@mrlubos mrlubos commented Mar 4, 2026

Closes #3494

@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@changeset-bot
Copy link

changeset-bot bot commented Mar 4, 2026

🦋 Changeset detected

Latest commit: ea2538b

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

This PR includes changesets to release 1 package
Name Type
@hey-api/openapi-ts Patch

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

@vercel
Copy link

vercel bot commented Mar 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hey-api-docs Ready Ready Preview, Comment Mar 4, 2026 11:16pm

Request Review

@mrlubos mrlubos marked this pull request as ready for review March 4, 2026 23:16
@pullfrog
Copy link

pullfrog bot commented Mar 4, 2026

Error

agent completed without reporting progress

Pullfrog  | Rerun failed job ➔View workflow run | Triggered by Pullfrogpullfrog.com𝕏

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. bug 🔥 Broken or incorrect behavior. labels Mar 4, 2026
@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 40.02%. Comparing base (d2c3705) to head (ea2538b).
⚠️ Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3498      +/-   ##
==========================================
- Coverage   40.24%   40.02%   -0.22%     
==========================================
  Files         484      486       +2     
  Lines       17610    17720     +110     
  Branches     5331     5361      +30     
==========================================
+ Hits         7087     7093       +6     
- Misses       8461     8542      +81     
- Partials     2062     2085      +23     
Flag Coverage Δ
unittests 40.02% <100.00%> (-0.22%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 4, 2026

Open in StackBlitz

@hey-api/codegen-core

npm i https://pkg.pr.new/@hey-api/codegen-core@3498

@hey-api/json-schema-ref-parser

npm i https://pkg.pr.new/@hey-api/json-schema-ref-parser@3498

@hey-api/nuxt

npm i https://pkg.pr.new/@hey-api/nuxt@3498

@hey-api/openapi-ts

npm i https://pkg.pr.new/@hey-api/openapi-ts@3498

@hey-api/shared

npm i https://pkg.pr.new/@hey-api/shared@3498

@hey-api/types

npm i https://pkg.pr.new/@hey-api/types@3498

@hey-api/vite-plugin

npm i https://pkg.pr.new/@hey-api/vite-plugin@3498

commit: e9f8bd0

@mrlubos mrlubos merged commit f4ab416 into main Mar 4, 2026
11 of 17 checks passed
@mrlubos mrlubos deleted the fix/dev-mode-check branch March 4, 2026 23:25
@hey-api hey-api bot mentioned this pull request Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🔥 Broken or incorrect behavior. size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve isDevMode check

1 participant