Thanks to visit codestin.com
Credit goes to github.com

Skip to content

chore(docs): add all Markdown files to docs app #1436

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 109 commits into from
Oct 17, 2022

Conversation

ChristopherPHolder
Copy link
Contributor

@ChristopherPHolder ChristopherPHolder commented Oct 12, 2022

Description

  • Move all Markdown files to the docs project.
  • Transclude CONTRIBUTING.md and package README.mds
  • List transcluded Markdown files as implicit Nx dependencies for the docs project

Closes #1430.

@nx-cloud
Copy link

nx-cloud bot commented Oct 12, 2022

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 5b67a3a. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

πŸ“‚ See all runs for this branch


βœ… Successfully ran 4 targets

Sent with πŸ’Œ from NxCloud.

@github-actions github-actions bot added </> Template @rx-angular/template related { } State @rx-angular/state related πŸ“– Docs API Documentation in the source code labels Oct 12, 2022
@ChristopherPHolder ChristopherPHolder marked this pull request as ready for review October 12, 2022 16:34
@LayZeeDK LayZeeDK linked an issue Oct 12, 2022 that may be closed by this pull request
3 tasks
@LayZeeDK LayZeeDK self-requested a review October 12, 2022 19:32
@LayZeeDK LayZeeDK self-assigned this Oct 12, 2022
Copy link
Member

@LayZeeDK LayZeeDK left a comment

Choose a reason for hiding this comment

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

@ChristopherPHolder
Please review my changes:

  • Move and include remaining libs/**/docs/Readme.md files
  • Correct Markdown outlines for proper tables of contents
  • Correct image embed URLs
  • Use Sentence case for titles and headings
  • Simplify navigation structure and configuration
  • Order State navigation
  • Update ESLint plugin's docsUrl function
  • Configure API reference slugs

Copy link
Contributor Author

@ChristopherPHolder ChristopherPHolder left a comment

Choose a reason for hiding this comment

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

@LayZeeDK All changes have been reviewed, I left a couple of questions.


Should consider unifying the approach we use to link internal MD files?

Markdown links

Examples of different linking styles in the repo:

  • In docs/cdk/notifications/_docs.md (65) -> rxLet
  • In docs/state/api/rx-state.md (64) -> stateful

@LayZeeDK LayZeeDK merged commit 7280626 into rx-angular:main Oct 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
πŸ› οΈ CDK CDK related πŸ“– Docs API Documentation in the source code πŸ“š Docs Web Documentation hosted on github pages { } State @rx-angular/state related </> Template @rx-angular/template related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

docs: add all Markdown documents
2 participants