Skip to content

Issue/309 docs table of contents#333

Merged
Eomm merged 6 commits intonodejs:masterfrom
thescientist13:issue/309-docs-table-of-contents
May 30, 2020
Merged

Issue/309 docs table of contents#333
Eomm merged 6 commits intonodejs:masterfrom
thescientist13:issue/309-docs-table-of-contents

Conversation

@thescientist13
Copy link
Copy Markdown
Contributor

@thescientist13 thescientist13 commented Mar 29, 2020

Description

Coming out of the 2020-03-10 meeting of the Package Maintenance team, on the topic of issue #309, the suggestion was that this comment on the suggestion of a Table of Contents for the docs in the repository be extracted out into its own PR.

Summary of Changes

  1. Updated the docs/README.md to provide additional context and motivation on behalf of the team, and created a ToC that links to existing docs.
  2. Created "stubs" for missing docs, that could become their own issues to track and can be authored independently and eventually promoted out of draft status.

Questions / Considerations

Just around docs in general...

  1. Is there a threshold for a doc moving out of draft status?
  2. Would there be any value in having a general format / convention around docs in terms of authoring / organization / structure?

Comment thread docs/drafts/worflows.md Outdated
Comment thread docs/drafts/code-of-conduct.md Outdated
Copy link
Copy Markdown
Contributor Author

@thescientist13 thescientist13 left a comment

Choose a reason for hiding this comment

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

Made follow up issues for the following topics in the list, either because they are new, or because it would be good to move them out of draft status:

  • Repository Settings (new) - #339
  • Continuous Integration / Delivery (draft) - #340
  • Workflows (draft) - #341
  • Code of Conduct (draft) - #342
  • Security (draft) - #343
  • Testing (draft) - #344
  • Publishing Guidelines (draft) - #345
  • Support (draft) - #346
  • Tools (draft) - #347

Depending on order of merges, we should probably include Dependency Management on this list as there is a PR open for adding it to the project.

Also missed out on tools.md, so added that to the ToC.


One question: is there criteria documented for what is considered a document in draft status vs not in draft status? In other words, how does a draft actually move out of that status, if it is in it?

@thescientist13 thescientist13 added the package-maintenance-agenda Agenda items for package-maintenance team label May 13, 2020
Copy link
Copy Markdown
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread docs/README.md
Comment thread docs/README.md Outdated
Comment thread docs/drafts/workflows.md Outdated
Copy link
Copy Markdown
Member

@BethGriggs BethGriggs left a comment

Choose a reason for hiding this comment

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

LGTM

thescientist13 and others added 2 commits May 20, 2020 09:54
Co-authored-by: Steven <steven@ceriously.com>
Co-authored-by: Steven <steven@ceriously.com>
@Eomm
Copy link
Copy Markdown
Member

Eomm commented May 30, 2020

Merging

more than 7 days from 4th approve and no blocking review

One question: is there criteria documented for what is considered a document in draft status vs not in draft status? In other words, how does a draft actually move out of that status, if it is in it?

I think we could could think to drafts as v0.x and not major released yet 😅 Usually it means that there is still discussion on them and those docs did not receive a complete consensus.

Would there be any value in having a general format / convention around docs in terms of authoring / organization / structure?

Right now there is not a shared structure (like file name in UPPERCASE and some in lower), it would be good have it since we are writing a lot of docs 👍

@Eomm Eomm merged commit cc04766 into nodejs:master May 30, 2020
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.

6 participants