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

Fix img link and adjust text #584

Merged
merged 2 commits into from
Oct 28, 2023
Merged

Fix img link and adjust text #584

merged 2 commits into from
Oct 28, 2023

Conversation

theDrake
Copy link
Contributor

One of the image links to a YouTube video wasn't properly formatted, so I fixed that and also went through the entire README making recommended adjustments to the text.

Some of my adjustments are important (in my opinion), others are just a matter of taste. Regardless, please feel free to reject any of them that don't appeal to you. If you're deeply curious as to why I made a particular change, feel free to ask and I'll attempt to explain my reasoning. Thank you for creating this tool and for considering my "improvements" to the README!

One of the image links to a YouTube video wasn't properly formatted, so I fixed that and also went through the entire README making recommended adjustments to the text.
@AdnanHodzic
Copy link
Owner

LGTM! Re-read through all of it I agree with all of it :)

Could you please resolve the conflict (new changes were made to README.md) and I'll merge your changes with master.

@theDrake
Copy link
Contributor Author

Glad to hear you liked the changes! I've resolved the conflict, so this should be good to go.

@AdnanHodzic
Copy link
Owner

Thank you for your contribution, you will be credited for your work as part of future release.

@AdnanHodzic AdnanHodzic merged commit 929fdf6 into AdnanHodzic:master Oct 28, 2023
@theDrake theDrake deleted the patch-1 branch October 29, 2023 19:08
shadeyg56 pushed a commit to shadeyg56/auto-cpufreq that referenced this pull request Feb 12, 2024
One of the image links to a YouTube video wasn't properly formatted, so I fixed that and also went through the entire README making recommended adjustments to the text.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants