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

chore: Update README with new setup instructions #172

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

chookity-pokk
Copy link
Collaborator

This is mentioned in #154 and I figured it was probably a good idea to get this in the official README. All credit goes to
@Arnavion

@chookity-pokk
Copy link
Collaborator Author

Closes #154

@chookity-pokk
Copy link
Collaborator Author

Since this is just a documentation change I am going to merge without review.

@chookity-pokk chookity-pokk merged commit adef634 into master Oct 2, 2024
8 checks passed
@Arnavion
Copy link

Arnavion commented Oct 2, 2024

You may want to keep the code fences because they make it clear where a command or command arg starts and ends.

@chookity-pokk
Copy link
Collaborator Author

You may want to keep the code fences because they make it clear where a command or command arg starts and ends.

@Arnavion Well shucks, looks like I should have waited for a review, so embarrassing. Thanks, I'll open a PR later adding those in.

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.

2 participants