ts-safeql / safeql

Validate and auto-generate TypeScript types from raw SQL queries in PostgreSQL.
https://safeql.dev
MIT License
1.35k stars 22 forks source link

Move @typescript-eslint/utils from deps to peer deps #259

Closed karlhorky closed 2 months ago

karlhorky commented 2 months ago

Closes #258

@Newbie012 what do you think about allowing projects to specify their version of @typescript-eslint/utils like this?

vercel[bot] commented 2 months ago

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

1 Skipped Deployment | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **safeql** | ⬜️ Ignored ([Inspect](https://vercel.com/newbie012s-projects/safeql/HwR9hnk3Za9rtsTUSkbrd8AnQPeA)) | [Visit Preview](https://safeql-git-move-typescript-eslint-ut-dc3a7c-newbie012s-projects.vercel.app) | | Sep 8, 2024 2:08pm |
changeset-bot[bot] commented 2 months ago

⚠️ No Changeset found

Latest commit: c6811b5f99a74525126b93fc329d97efc1f81f0b

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

Newbie012 commented 2 months ago

I don't mind. make sure that pnpm-lock.yml is updated as well.

karlhorky commented 2 months ago

Great, updated the lockfile and updated to pnpm 9.9.0

karlhorky commented 2 months ago

Thanks for the review and merge! I guess this will be released soon in the next patch or minor?

Newbie012 commented 2 months ago

yes. write a changeset as patch.

karlhorky commented 2 months ago

Done, PR open: