Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

docs: simplify install docs#11946

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
bpmct merged 5 commits intomainfrominstall-docs
Mar 8, 2024
Merged

docs: simplify install docs#11946

bpmct merged 5 commits intomainfrominstall-docs
Mar 8, 2024

Conversation

bpmct
Copy link
Member

@bpmctbpmct commentedJan 30, 2024•
edited
Loading

While we support a diverse amount of installation methods, only power users will want to use a system service/non-install script. Folks who want to run a system service on a full-time VM should understand how to start it and keep it running. We can always make a guide/tutorial for this that walks through the basics

BeforeAfter
Screenshot 2024-01-30 at 4 18 30 PMimage

Main concerns with the old one:

  • Nested tabs
  • Too many decisions for a novice user

stirby reacted with thumbs up emoji
matifali

This comment was marked as outdated.

@matifalimatifali self-requested a reviewJanuary 30, 2024 22:39
Copy link
Member

@matifalimatifali left a comment•
edited
Loading

Choose a reason for hiding this comment

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

I am ok with this refactoring but we are now removing all the other installation methods we support. We should have them somewhere for the people who need.

Probably the full original detailed installation methods could be moved to a tutorial or guide in the guides section.
I don't think we should just delete it.

Could you copy the existing install sh as is to a new guide with a title "How to install Coder?"

Copy link
Member

@matifalimatifali left a comment

Choose a reason for hiding this comment

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

I revisited and agree with the refactoring. A few suggestions regarding wrong/old links and it's good to merge.

@matifalimatifali added the docsArea: coder.com/docs labelJan 31, 2024
@matifali
Copy link
Member

@bpmct, do you want to make any further changes before merging?

@bpmct
Copy link
MemberAuthor

Yeah, I have several changes planned! Should have the rest pushed soon

matifali reacted with thumbs up emoji

@github-actionsgithub-actionsbot added the staleThis issue is like stale bread. labelFeb 14, 2024
@matifali
Copy link
Member

@bpmct I think you still want to complete this.

@matifalimatifali reopened thisFeb 18, 2024
@github-actionsgithub-actionsbot removed the staleThis issue is like stale bread. labelFeb 19, 2024
@github-actionsgithub-actionsbot added the staleThis issue is like stale bread. labelFeb 27, 2024
@bpmctbpmct reopened thisMar 7, 2024
@bpmctbpmctenabled auto-merge (squash)March 7, 2024 23:34
@github-actionsgithub-actionsbot removed the staleThis issue is like stale bread. labelMar 8, 2024
@bpmctbpmct merged commit1e17782 intomainMar 8, 2024
@bpmctbpmct deleted the install-docs branchMarch 8, 2024 15:16
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsMar 8, 2024
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers

@matifalimatifalimatifali approved these changes

@kylecarbskylecarbsAwaiting requested review from kylecarbs

Assignees

@bpmctbpmct

Labels
docsArea: coder.com/docs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@bpmct@matifali
[8]ページ先頭
©2009-2025 Movatter.jp