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

Update dendron.topic.publish.cook.github-action.action-detail.md #699

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RaneyDazed
Copy link

@RaneyDazed RaneyDazed commented Apr 14, 2023

What does this PR do?

fix spelling error on line 25 dnedron and line 12 frist

I may be missing something here, the repos I've contributed to most don't have many requirements for updating docs.

What issues does this PR fix or reference?

NOTE: Fixes will close the ticket after merging. Relates to: will leave the ticket open after merging.

Merge requirements satisfied

For more information, visit the Contributing Guide for Documentation

  • Check rendered output to ensure formatting is correct for renderings of wikilinks, note refs, tables, and images. Dendron: Show Preview can be used in your workspace to confirm the page renders as expected. Sometimes, Dendron: Reload Index needs to be ran if certain wikilinks aren't working as expected. (ignore this if contribution is made via the Edit this page on GitHub button from the published site)
  • If this PR includes any refactoring (renaming notes, renaming headers, etc.), make sure that these changes were done via Dendron Refactoring Commands. This will ensure that any other impacted areas of the documentation, that link to or embed modified notes (via note references), are automatically updated.

Verify GitHub Actions tests are passing, which can be seen in the Checks tab of the PR:

  • URL validator GitHub Action passes
  • Dendron site build GitHub Action passes

NOTE: If you are a new contributor, a maintainer will need to provide approval for GitHub Actions to run on your PRs.

fix spelling error on line 25 `dnedron` and line 12 `frist`
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.

None yet

1 participant