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

docs: slim down readme #2140

Merged
merged 6 commits into from Jun 8, 2022
Merged

docs: slim down readme #2140

merged 6 commits into from Jun 8, 2022

Conversation

jsjoeio
Copy link
Contributor

@jsjoeio jsjoeio commented Jun 7, 2022

Description

This PR refactors the README.md to be more slim and concise with the goal of making it less overwhelming when someone lands on the repo page.

I also moved the Documentation Table of Contents to ./docs/index.md and some getting started information to ./docs/walkthrough.md. Open to suggestions!

🖼️ Rendered

@jsjoeio jsjoeio self-assigned this Jun 7, 2022
@jsjoeio jsjoeio added the docs Area: docs label Jun 7, 2022

> **Note**:
> Coder is in an alpha state, but any serious bugs are P1 for us so [please report them](https://github.com/coder/coder/issues/new/choose).
Coder creates remote development machines so your team can develop from anywhere.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Keep it short and simple.


## Installing Coder
> **Note**:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Important, but they don't need to know this until they go to install and use it.

**Manage less**

- Ensure your entire team is using the same tools and resources
- Rollout critical updates to your developers with one command
- Automatically shut down expensive cloud resources
- Keep your source code and data behind your firewall

## How it works
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Diff is hard to read. I just moved these down a bit.

Join the community on [Discord](https://discord.gg/coder) and [Twitter](https://twitter.com/coderhq) #coder!

[Suggest improvements and report problems](https://github.com/coder/coder/issues/new/choose)
Visit our docs [here](./docs/index.md).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I called it index.md because that was what first came to mind. Open to other suggestions.

@@ -155,7 +72,13 @@ Please file [an issue](https://github.com/coder/coder/issues/new) if any informa

---

_As of 5/27/22_
_Last updated: 5/27/22_
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Felt like this was easier to understand

docs/walkthrough.md Outdated Show resolved Hide resolved
Copy link
Member

@bpmct bpmct left a comment

Choose a reason for hiding this comment

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

All these changes look good to me, I added some suggestions.

docs/walkthrough.md Outdated Show resolved Hide resolved
docs/index.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Comment on lines -58 to -60
<p align="center">
<img src="./docs/images/ide-icons.svg" height=72>
</p>
Copy link
Member

Choose a reason for hiding this comment

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

While it does take up some scrolling space, we've received some early feedback that the IDE/cloud icons helped explain what Coder does. Do you think these make the README more confusing?

Copy link
Member

@bpmct bpmct Jun 8, 2022

Choose a reason for hiding this comment

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

hmm... When we use a product screenshot, we can attempt to include the use of the IDE icons and clouds there

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I like the second idea!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we've received some early feedback that the IDE/cloud icons helped explain what Coder does

I agree with this, but it also makes me wonder, does our copy not convey this already? It probably should! aka the IDE/Cloud icons should be additive and shouldn't be necessary to explain Coder.

@jsjoeio jsjoeio marked this pull request as ready for review June 8, 2022 15:15
@jsjoeio jsjoeio merged commit b6899e2 into main Jun 8, 2022
@jsjoeio jsjoeio deleted the jsjoeio/slim-readme branch June 8, 2022 15:37
kylecarbs pushed a commit that referenced this pull request Jun 10, 2022
* docs: move docs table of contents to docs/index.md

* docs: move how it works and IDE support to about.md

* docs: move readme steps to walkthrough

* docs: slim down readme

* refactor: walkthrough -> quickstart

* docs: minor edits
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Area: docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants