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

ReadMe Fixes #2083

Merged
merged 3 commits into from
Nov 22, 2024
Merged

ReadMe Fixes #2083

merged 3 commits into from
Nov 22, 2024

Conversation

stefankiesz
Copy link
Contributor

Issue #, if available:

What was changed?

  • Merging some ReadMe fixes that were already merged into the old develop branch.

Why was it changed?

  • To bring in minor changes from old develop branch.

How was it changed?

  • By cherry-picking the commits.

What testing was done for the changes?

  • Proof-reading the readme changes.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

* Add copy code buttons to github clone commands

* Add language specifier
README.md Show resolved Hide resolved
@stefankiesz stefankiesz merged commit 245e74d into develop Nov 22, 2024
28 checks passed
@stefankiesz stefankiesz deleted the old-develop-changes branch November 22, 2024 20:56
@sirknightj
Copy link
Contributor

Just noticed, the build command will need to take the folder name into account

@stefankiesz
Copy link
Contributor Author

Nice catch, adding that fix to the other ReadMe PR.

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.

3 participants