Skip to content
This repository has been archived by the owner on Jul 7, 2024. It is now read-only.

Fix: outdated doccumentation #49

Merged
merged 5 commits into from
Nov 10, 2023
Merged

Conversation

Zephira58
Copy link
Contributor

There was a minor issue with the dev environment doc that caused me some minor headaches when i first joined the project, I've gone ahead and updated it accordingly

Copy link
Collaborator

@Cpt-Dingus Cpt-Dingus left a comment

Choose a reason for hiding this comment

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

I think changing it to the following is a bit clearer

docs/guides/dev_environment.md Outdated Show resolved Hide resolved
docs/guides/dev_environment.md Outdated Show resolved Hide resolved
docs/guides/dev_environment.md Outdated Show resolved Hide resolved
docs/guides/dev_environment.md Outdated Show resolved Hide resolved
@Cpt-Dingus
Copy link
Collaborator

had some mistakes in my previous review, my bad

@Zephira58
Copy link
Contributor Author

had some mistakes in my previous review, my bad

No stress, it was like 1am when i looked at your suggestions too

Copy link
Collaborator

@Cpt-Dingus Cpt-Dingus left a comment

Choose a reason for hiding this comment

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

LGTM

@Cpt-Dingus Cpt-Dingus merged commit e75cab7 into r-Techsupport:main Nov 10, 2023
1 check passed
@Zephira58 Zephira58 deleted the doc-fix branch November 11, 2023 02:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants