Skip to content

fix(cli): prisma version --json suppress dotenv message#25535

Merged
jkomyno merged 1 commit intoprisma:mainfrom
hinaloe:fix/25478-version-ignore-message
Oct 30, 2024
Merged

fix(cli): prisma version --json suppress dotenv message#25535
jkomyno merged 1 commit intoprisma:mainfrom
hinaloe:fix/25478-version-ignore-message

Conversation

@hinaloe
Copy link
Copy Markdown
Contributor

@hinaloe hinaloe commented Oct 29, 2024

prisma version --json should not contain non-json string.

- fix prisma#25478

`prisma version --json` should not contain non-json string.
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Oct 29, 2024
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Oct 29, 2024

CLA assistant check
All committers have signed the CLA.

@jkomyno jkomyno added this to the 5.22.0 milestone Oct 30, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 30, 2024
@jkomyno
Copy link
Copy Markdown
Contributor

jkomyno commented Oct 30, 2024

Thanks @hinaloe!

@jkomyno jkomyno merged commit 738d5b7 into prisma:main Oct 30, 2024
@hinaloe hinaloe deleted the fix/25478-version-ignore-message branch October 30, 2024 12:57
jkomyno added a commit that referenced this pull request Oct 31, 2024
* feat(cli): [BREAKING] remove custom yarn logic

* test(internals): update snapshots

* chore(internals): get rid of unused fixtures

* fix(fetch-engine): use rename syscall to ensure atomic moves (#25129)

* fix(adapter-d1): throws Error when string column contains any ISO date (#25483)

* fix(cli): `prisma version --json` suppress dotenv message (#25535)

- fix #25478

`prisma version --json` should not contain non-json string.

* chore: removed unused file

* chore: restore yarn detection logic for Windows

---------

Co-authored-by: Aman Karmani <aman@tmm1.net>
Co-authored-by: Yamashiro Takuro <39460271+Takur0@users.noreply.github.com>
Co-authored-by: Hinaloe <hina@hinaloe.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: prisma version --json should suppress dotenv message

3 participants