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: check-types in kitchen-sink example #9952

Merged
merged 1 commit into from
Feb 14, 2025

Conversation

Asupkay
Copy link
Contributor

@Asupkay Asupkay commented Feb 13, 2025

Description

The kitchen-sink example currently has its check types command set up as "typecheck": "turbo run typecheck" however none of its packages or apps have it set up that way instead utilizing check-types. This causes the following error where none of the type checking jobs run:

image

This pull request changes the root package.json's check types command to check-types so that all the packages and apps correctly run type checking. This also matches other examples such as basic and with-tailwind.

Testing Instructions

To test open up the examples/kitchen-sink folder and run yarn check-types you will see the type checking jobs successfully run.
image

@Asupkay Asupkay requested a review from a team as a code owner February 13, 2025 05:52
@turbo-orchestrator turbo-orchestrator bot added area: examples Improvements or additions to examples needs: triage New issues get this label. Remove it after triage labels Feb 13, 2025
Copy link

vercel bot commented Feb 13, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
examples-basic-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 3:18am
examples-designsystem-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 3:18am
examples-gatsby-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 3:18am
examples-kitchensink-blog ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 3:18am
examples-native-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 3:18am
examples-nonmonorepo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 3:18am
examples-svelte-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 3:18am
examples-tailwind-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 3:18am
examples-vite-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 3:18am

Copy link

vercel bot commented Feb 13, 2025

@Asupkay is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@Asupkay Asupkay changed the title Fix check-types in kitchen-sink fix: check-types in kitchen-sink example Feb 13, 2025
Copy link
Contributor

@anthonyshew anthonyshew left a comment

Choose a reason for hiding this comment

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

Oops, thank you!

@anthonyshew anthonyshew merged commit 9f79607 into vercel:main Feb 14, 2025
36 of 37 checks passed
joshnuss pushed a commit to joshnuss/turborepo that referenced this pull request Feb 15, 2025
### Description

The kitchen-sink example currently has its check types command set up as
"typecheck": "turbo run typecheck" however none of its packages or apps
have it set up that way instead utilizing check-types. This causes the
following error where none of the type checking jobs run:


![image](https://github.com/user-attachments/assets/e38cc1cf-8144-435d-a2b3-55e4cd0425c0)

This pull request changes the root package.json's check types command to
check-types so that all the packages and apps correctly run type
checking. This also matches other examples such as basic and
with-tailwind.

### Testing Instructions
To test open up the examples/kitchen-sink folder and run yarn
check-types you will see the type checking jobs successfully run.

![image](https://github.com/user-attachments/assets/ceb42d4e-8e85-4aa5-8a66-de357eac42f7)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area: examples Improvements or additions to examples needs: triage New issues get this label. Remove it after triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants