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

Fixes various errors in the documentation #693

Merged
merged 6 commits into from
Oct 14, 2023

Conversation

bnonni
Copy link
Contributor

@bnonni bnonni commented Oct 11, 2023

Overview

There's a handful of places in the docs that go to dead links or paths.

Description

Wanted to update those docs to help others in the future.

How Has This Been Tested?

Since this is a purely markdown and comment change, I didn't write tests. Please let me know if I need to do that, as I'm happy to do so.

Checklist

Tagging @decentralgabe @andresuribe87 since I've been communicating with them regarding these changes
Before submitting this PR, please make sure:

  • I have read the CONTRIBUTING document.
  • My code is consistent with the rest of the project
  • I have tagged the relevant reviewers and/or interested parties
  • I have updated the READMEs and other documentation of affected packages

References

Docs changed:

fixes various doc mds

more additions to docs

more edits to docs
@bnonni bnonni force-pushed the bug/documentation branch from de420a9 to 979871f Compare October 11, 2023 06:34
Copy link
Contributor

@andresuribe87 andresuribe87 left a comment

Choose a reason for hiding this comment

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

Thanks so much! Couple of comments.

config/.env.example Show resolved Hide resolved
doc/config/toml.md Outdated Show resolved Hide resolved
Copy link
Member

@decentralgabe decentralgabe left a comment

Choose a reason for hiding this comment

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

thank you

@bnonni
Copy link
Contributor Author

bnonni commented Oct 13, 2023

Thanks so much! Couple of comments.

Comments replied to. Changes pushed.

thank you

you're welcome. happy to help.

@andresuribe87 andresuribe87 merged commit 690a6f9 into TBD54566975:main Oct 14, 2023
6 checks passed
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.

3 participants