X Tutup
The Wayback Machine - https://web.archive.org/web/20210817091212/https://github.com/prisma/prisma/issues/8063
Skip to content

/ prisma

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Drift detection makes CLI output where bold #8063

Open
janpio opened this issue Jul 4, 2021 · 3 comments
Open

Drift detection makes CLI output where bold #8063

janpio opened this issue Jul 4, 2021 · 3 comments
Labels
good first issue kind/improvement team/migrations tech/typescript topic: cli-dev topic: cli-migrate topic: cli topic: drift topic: migrate dev topic: migrate

Comments

@janpio
Copy link
Member

@janpio janpio commented Jul 4, 2021

image

Intended that the full output is bolded?
@tomhoule
Copy link
Member

@tomhoule tomhoule commented Jul 5, 2021

That's a question for @Jolg42 — I like the attention drawing bold font for dev diagnostics personally, but I don't know if it's intentional.
@Jolg42
Copy link
Member

@Jolg42 Jolg42 commented Jul 6, 2021

The bold text here is from devDiagnostic.action.reason
Screen Shot 2021-07-06 at 17 50 23

It's printed as part of the prompt here

prisma/src/packages/migrate/src/commands/MigrateDev.ts

Line 322 in 15e1ca3

const message = `${reason}

From what I see the bold comes from passing the text to the "prompts" lib which adds bold to all the text passed.

So not intended I think. We could print the reason independently as console.info and then do the prompt.
@janpio
Copy link
Member Author

@janpio janpio commented Jul 9, 2021

Similar is true for the reset:
image
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue kind/improvement team/migrations tech/typescript topic: cli-dev topic: cli-migrate topic: cli topic: drift topic: migrate dev topic: migrate
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
@janpio @thebiglabasky @Jolg42 @tomhoule @pantharshit00
X Tutup