Skip to content

docs: Move README to /docs #871

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 1 commit into from
Apr 4, 2022
Merged

docs: Move README to /docs #871

merged 1 commit into from
Apr 4, 2022

Conversation

jsjoeio
Copy link
Contributor

@jsjoeio jsjoeio commented Apr 4, 2022

This moves the README.md to a new top-level folder called /docs. We plan to add more Markdown files so this will prevent us from "polluting" the top-level space with doc files.

Will this still show on the GitHub homepage?

Yup!

If you put your README file in your repository's root, docs, or hidden .github directory, GitHub will recognize and automatically surface your README to repository visitors.

Source

This is how we do it in code-server as well.

Fixes #654

@jsjoeio jsjoeio added the docs Area: coder.com/docs label Apr 4, 2022
@jsjoeio jsjoeio self-assigned this Apr 4, 2022
@codecov
Copy link

codecov bot commented Apr 4, 2022

Codecov Report

Merging #871 (1893c45) into main (63b4006) will increase coverage by 0.23%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #871      +/-   ##
==========================================
+ Coverage   65.65%   65.89%   +0.23%     
==========================================
  Files         203      203              
  Lines       13226    13226              
  Branches       86       86              
==========================================
+ Hits         8684     8715      +31     
+ Misses       3645     3622      -23     
+ Partials      897      889       -8     
Flag Coverage Δ
unittest-go- 65.18% <ø> (+0.14%) ⬆️
unittest-go-macos-latest 52.72% <ø> (+0.22%) ⬆️
unittest-go-ubuntu-latest 55.18% <ø> (+0.09%) ⬆️
unittest-go-windows-2022 51.81% <ø> (+0.04%) ⬆️
unittest-js 62.70% <ø> (ø)
Impacted Files Coverage Δ
provisioner/echo/serve.go 54.40% <0.00%> (-4.81%) ⬇️
coderd/provisionerdaemons.go 59.02% <0.00%> (+0.50%) ⬆️
provisionerd/provisionerd.go 81.35% <0.00%> (+1.02%) ⬆️
peer/conn.go 80.96% <0.00%> (+3.04%) ⬆️
coderd/workspaceresources.go 59.34% <0.00%> (+3.25%) ⬆️
cli/cliui/agent.go 82.45% <0.00%> (+5.26%) ⬆️
peerbroker/dial.go 83.60% <0.00%> (+6.55%) ⬆️

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 63b4006...1893c45. Read the comment docs.

Copy link
Member

@kylecarbs kylecarbs left a comment

Choose a reason for hiding this comment

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

Seems reasonable to me!

@kylecarbs
Copy link
Member

Could we remove the (README) in the title? I feel like docs: Move README to /docs reads a bit better!

@jsjoeio jsjoeio changed the title docs(README): move to /docs folder docs: Move README to /docs Apr 4, 2022
@jsjoeio
Copy link
Contributor Author

jsjoeio commented Apr 4, 2022

Could we remove the (README) in the title? I feel like docs: Move README to /docs reads a bit better!

Sure! (P.S. in the future, feel free to edit titles if you see a better fit - I don't mind and I trust ya!)

@jsjoeio jsjoeio merged commit f02b8fd into main Apr 4, 2022
@jsjoeio jsjoeio deleted the jsjoeio/add-docs branch April 4, 2022 22:48
@misskniss misskniss added this to the V2 Beta milestone May 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Area: coder.com/docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docs folder at root with markdown
3 participants