Skip to content

Conversation

trivikr
Copy link
Member

@trivikr trivikr commented Jul 28, 2024

Issue

Missed while migrating to biome in #886

Description

Removes prettier API call while generating client types map

Testing

Generated client types map by running

$ yarn tsx scripts/generateClientTypesMap/index.ts

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Copy link

changeset-bot bot commented Jul 28, 2024

⚠️ No Changeset found

Latest commit: d10644e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@trivikr trivikr force-pushed the scripts-remove-prettier branch from 6aa7435 to d1044a6 Compare July 28, 2024 02:16
@trivikr

This comment was marked as outdated.

@trivikr trivikr changed the title Remove prettier API call while generating client types map Use biome CLI for formatting client types map Jul 28, 2024
@trivikr trivikr marked this pull request as ready for review July 28, 2024 02:35
@trivikr trivikr requested a review from a team as a code owner July 28, 2024 02:35
@trivikr trivikr merged commit a777b79 into main Jul 28, 2024
6 checks passed
@trivikr trivikr deleted the scripts-remove-prettier branch July 28, 2024 02:35
Copy link
Contributor

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant