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(readme): various updates for clarity #171

Merged
merged 2 commits into from
Feb 15, 2023

Conversation

BigLep
Copy link
Contributor

@BigLep BigLep commented Feb 14, 2023

Moved CODEOWNERS under docs for discoverability
Added @ipfs/kubo-maintainters as default codeowner for clarity
Added TOC
Added more notes on motivation and scope.
Linked to examples.

Moved CODEOWNERS under docs for discoverability
Added @ipfs/kubo-maintainters as default codeowner for clarity
Added TOC
Added more notes on motivation and scope.
Linked to examples.
@ipfs ipfs deleted a comment from welcome bot Feb 14, 2023
@codecov
Copy link

codecov bot commented Feb 14, 2023

Codecov Report

Merging #171 (3413ce3) into main (2902465) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #171   +/-   ##
=======================================
  Coverage   19.62%   19.62%           
=======================================
  Files         100      100           
  Lines       11053    11053           
=======================================
  Hits         2169     2169           
  Misses       8597     8597           
  Partials      287      287           

@lidel lidel changed the title Various README updates for clarity docs(readme): various updates for clarity Feb 14, 2023
docs/CODEOWNERS Show resolved Hide resolved
@guseggert guseggert dismissed Jorropo’s stale review February 15, 2023 14:00

addressed w/o code change

@guseggert guseggert merged commit d195f6d into main Feb 15, 2023
@guseggert guseggert deleted the chore/biglep-readme-updates-2023-02-13 branch February 15, 2023 14:01
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.

4 participants