Skip to content

Conversation

@Jak-Ch-ll
Copy link
Contributor

@Jak-Ch-ll Jak-Ch-ll commented Oct 12, 2025

In the current setup Vitest does not report type errors in the type test file, so the file will always succeed. Issues would still be picked up by tsc so it isn't the worst, but my expectation would be use Vitests type testing correctly. Especially when working on some types (which I might be doing right now 😄) Vitest can give quicker feedback.

I changed the setup a bit to enable this which also has the upside that you can run pnpm test --project X depending on what you are currently working.

@nserver-dokploy
Copy link

🚨 Preview Deployment Blocked - Security Protection

Your pull request was blocked from triggering preview deployments

Why was this blocked?

  • User: Jak-Ch-ll
  • Repository: nuxt-open-fetch
  • Permission Level: read
  • Required Level: write, maintain, or admin

How to resolve this:

Option 1: Get Collaborator Access (Recommended)
Ask a repository maintainer to invite you as a collaborator with write permissions or higher.

Option 2: Request Permission Override
Ask a repository administrator to disable security validation for this specific application if appropriate.

For Repository Administrators:

To disable this security check (⚠️ not recommended for public repositories):
Enter to preview settings and disable the security check.


This security measure protects against malicious code execution in preview deployments. Only trusted collaborators should have the ability to trigger deployments.

🛡️ Learn more about this security feature

This protection prevents unauthorized users from:

  • Executing malicious code on the deployment server
  • Accessing environment variables and secrets
  • Potentially compromising the infrastructure

Preview deployments are powerful but require trust. Only users with repository write access can trigger them.

Copy link
Owner

@Norbiros Norbiros left a comment

Choose a reason for hiding this comment

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

LGTM! BTW, thanks for contributing a lot to this project ❤️

@Norbiros Norbiros merged commit bc60e62 into Norbiros:main Oct 13, 2025
2 checks passed
@Jak-Ch-ll
Copy link
Contributor Author

LGTM! BTW, thanks for contributing a lot to this project ❤️

My pleasure and thank you for stepping in as maintainer and keeping the project alive!

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