Skip to content

Conversation

kylecarbs
Copy link
Member

Workspace commands will be aliased at the top-level, so
"start" would easily be confused with starting a workspace.

Server seems like a more appropriate name too.

@kylecarbs kylecarbs requested review from f0ssel and bpmct April 23, 2022 16:04
@kylecarbs kylecarbs self-assigned this Apr 23, 2022
@codecov
Copy link

codecov bot commented Apr 23, 2022

Codecov Report

Merging #1110 (265fb7b) into main (da36812) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1110      +/-   ##
==========================================
- Coverage   66.58%   66.58%   -0.01%     
==========================================
  Files         255      255              
  Lines       15743    15743              
  Branches      152      152              
==========================================
- Hits        10483    10482       -1     
- Misses       4190     4195       +5     
+ Partials     1070     1066       -4     
Flag Coverage Δ
unittest-go-macos-latest 53.39% <100.00%> (+0.01%) ⬆️
unittest-go-postgres- 65.90% <100.00%> (-0.03%) ⬇️
unittest-go-ubuntu-latest 56.04% <100.00%> (+0.02%) ⬆️
unittest-go-windows-2022 52.92% <100.00%> (+0.02%) ⬆️
unittest-js 66.83% <ø> (ø)
Impacted Files Coverage Δ
cli/root.go 77.27% <100.00%> (ø)
cli/server.go 62.04% <100.00%> (ø)
provisioner/echo/serve.go 56.80% <0.00%> (-2.41%) ⬇️
provisionerd/provisionerd.go 79.85% <0.00%> (-1.33%) ⬇️
coderd/provisionerdaemons.go 60.37% <0.00%> (ø)
peer/conn.go 79.44% <0.00%> (+0.25%) ⬆️
provisioner/terraform/provision.go 71.39% <0.00%> (+0.43%) ⬆️
codersdk/provisionerdaemons.go 65.67% <0.00%> (+2.98%) ⬆️
cli/cliui/agent.go 82.45% <0.00%> (+5.26%) ⬆️
provisionersdk/transport.go 85.10% <0.00%> (+6.38%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update da36812...265fb7b. Read the comment docs.

@kylecarbs kylecarbs requested a review from a team as a code owner April 23, 2022 16:51
@kylecarbs kylecarbs requested review from coadler and removed request for a team and f0ssel April 23, 2022 16:51
Workspace commands will be aliased at the top-level, so
"start" would easily be confused with starting a workspace.

Server seems like a more appropriate name too.
@kylecarbs kylecarbs merged commit 3976994 into main Apr 23, 2022
@kylecarbs kylecarbs deleted the mvstart branch April 23, 2022 17:19
kylecarbs added a commit that referenced this pull request Apr 23, 2022
Workspace commands will be aliased at the top-level, so
"start" would easily be confused with starting a workspace.

Server seems like a more appropriate name too.
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