Skip to content

feat: add version checking to CLI #2643

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.

Sign up for GitHub

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 18 commits into from
Jun 29, 2022
Merged

feat: add version checking to CLI #2643

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
fe44291
feat: add version checking to CLI
sreya Jun 24, 2022
a9b4796
stuff
sreya Jun 24, 2022
19224b0
add more plumbing
sreya Jun 24, 2022
ff7d39e
remove some test code
sreya Jun 24, 2022
aa1ee2f
remove unnecessary type cast
sreya Jun 24, 2022
2b52169
prevent duplicates
sreya Jun 24, 2022
5435418
make gen
sreya Jun 24, 2022
9a9bae1
use semver
sreya Jun 24, 2022
cc6baed
Revert "use semver"
sreya Jun 24, 2022
6e8c3c0
use install script link
sreya Jun 24, 2022
fc3fbb5
Merge branch 'main' into jon/version
sreya Jun 27, 2022
67966e7
some merge woes
sreya Jun 27, 2022
d855172
some stuff
sreya Jun 27, 2022
54cb664
more stuff
sreya Jun 28, 2022
51301d9
fix versions
sreya Jun 28, 2022
636d5db
remove race condition
sreya Jun 28, 2022
d36e134
make gen
sreya Jun 28, 2022
b07de24
pr comments
sreya Jun 29, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into jon/version
  • Loading branch information
@sreya
sreya committed Jun 27, 2022
commit fc3fbb59f7e2610f7e0a79b0e62746fbe07576de
5 changes: 3 additions & 2 deletions cli/root.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ const (
var (
errUnauthenticated = xerrors.New(notLoggedInMessage)
varSuppressVersion = false
envSessionToken = "CODER_SESSION_TOKEN"
)

func init() {
Expand Down Expand Up @@ -126,9 +127,9 @@ func Root() *cobra.Command {
cmd.SetUsageTemplate(usageTemplate())

cmd.PersistentFlags().String(varURL, "", "Specify the URL to your deployment.")
cmd.PersistentFlags().String(varToken, "", "Specify an authentication token.")
cliflag.BoolVarP(cmd.PersistentFlags(), &varSuppressVersion, "no-version-warning", "", envNoVersionCheck, false, "Suppress warning when client and server versions do not match.")
cliflag.String(cmd.PersistentFlags(), varAgentToken, "", "CODER_AGENT_TOKEN", "", "Specify an agent authentication token.")
=======
>>>>>>> main
_ = cmd.PersistentFlags().MarkHidden(varAgentToken)
cliflag.String(cmd.PersistentFlags(), varAgentURL, "", "CODER_AGENT_URL", "", "Specify the URL for an agent to access your deployment.")
_ = cmd.PersistentFlags().MarkHidden(varAgentURL)
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.