Skip to content
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

Adapt CLI update notifier for major upgrade #9061

Closed
janpio opened this issue Sep 1, 2021 · 0 comments · Fixed by #9079
Closed

Adapt CLI update notifier for major upgrade #9061

janpio opened this issue Sep 1, 2021 · 0 comments · Fixed by #9079
Assignees
@janpio @Jolg42
Labels
kind/improvement An improvement to existing feature and code. topic: cli topic: update-notifier
Milestone
2.31.0 / 3.0.x

Comments

@janpio
Copy link
Member

janpio commented Sep 1, 2021

Right now the update notifier in Prisma CLI is not optimal for major upgrades. We want to improve the experience there.

Context:

prisma/packages/cli/src/utils/printUpdateMessage.ts

Lines 120 to 122 in ba74c81

if (tag && tag !== 'latest') {
command += `@${tag}`
}

@janpio janpio added kind/improvement An improvement to existing feature and code. topic: cli topic: update-notifier labels Sep 1, 2021
@Jolg42 Jolg42 linked a pull request Sep 2, 2021 that will close this issue
feat(cli): major update msg + remove @prisma/cli update warning #9079
Merged
@Jolg42 Jolg42 added this to the 2.31.0 / 3.0.x milestone Sep 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@janpio janpio

@Jolg42 Jolg42

Labels
kind/improvement An improvement to existing feature and code. topic: cli topic: update-notifier
Projects
None yet
Milestone
2.31.0 / 3.0.x
Development

Successfully merging a pull request may close this issue.

feat(cli): major update msg + remove @prisma/cli update warning prisma/prisma
2 participants
@janpio @Jolg42