Skip to content

add warning for --link deprecation #18

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

Merged
merged 2 commits into from
Mar 21, 2022

Conversation

bpmct
Copy link
Member

@bpmct bpmct commented Nov 29, 2021

No description provided.

Comment on lines +116 to +120
flog.Info("code-server --link is deprecated. While the servers will remain online,")
flog.Info("we are not releasing new features or bugfixes. A future code-server")
flog.Info("release will include a v2 with new features. If you would")
flog.Info("like early access, reach out on https://cdr.co/join-community")
flog.Info("")
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was not sure if this was the best way to do multi-line messages with flog. I also noticed there is no "warning" level for logs (see coder/flog#1)

@jsjoeio
Copy link
Contributor

jsjoeio commented Mar 18, 2022

@bpmct is this something we should merge?

@bpmct
Copy link
Member Author

bpmct commented Mar 18, 2022

Yep, I think so. It has been deprecated in the code-server CLI for a while.

@bpmct bpmct requested a review from jsjoeio March 18, 2022 23:14
@jsjoeio jsjoeio merged commit 9d420dc into coder:master Mar 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants