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 README.md #185

Merged
merged 5 commits into from
Jan 21, 2025
Merged

Update README.md #185

merged 5 commits into from
Jan 21, 2025

Conversation

ChucklesOnGitHub
Copy link
Contributor

ran into some issues when installing and made updates for clarity.
Will read through section about best practices before pushing and then wrap up this pr.
Maybe add the Ctrl+Shift+V shortchut to pull up the reader in VSCode? I guess it was IDE agnostic but still helpful.

ran into some issues when installing and made updates for clarity
@ChucklesOnGitHub ChucklesOnGitHub self-assigned this Jan 15, 2025
@ChucklesOnGitHub
Copy link
Contributor Author

the pre-push checks seemed to run allright with these instructions. i ran all of them manually before realizing that at the end of the document was the one command to run them all, but whatevs. made me read the whole thing which is good.

@ChucklesOnGitHub ChucklesOnGitHub marked this pull request as ready for review January 17, 2025 18:17
@cjsha cjsha merged commit 3e3acf5 into main Jan 21, 2025
3 checks passed
@cjsha cjsha deleted the readme-update branch January 21, 2025 23:39
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