Skip to content

Fix for Docker with instructions in readme #38

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

saltedlolly
Copy link

@saltedlolly saltedlolly commented Aug 11, 2025

I can only apologise. In trying to separate the Docker changes into a separate PR I managed to accidentally add it to the other PR by mistake. Then in trying to fix it, I managed to make it worse. This PR is simply to fix the readme even though it includes other previous commits.

Copy link
Member

@Sorixelle Sorixelle left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for the fixes! I've split out the PRs for you, just need to fix the commit lints and we're good to merge.

@Sorixelle Sorixelle added type: bug Something isn't working area: docs Changes to documentation about working with the repo. NOT for changes to documentation site content! labels Aug 12, 2025
@saltedlolly
Copy link
Author

I am banging my head against the wall. I fixed the lint and it added back in the other commits that you removed.

@saltedlolly
Copy link
Author

I f**king fixed it! I finally figured out how to get rid of the other commits from this branch! It has taken me hours to understand this stuff! I am hoping that I am finally starting to understand rebase.

@saltedlolly saltedlolly changed the title Update docker instructions in readme Fix for Docker with instructions in readme Aug 12, 2025
@Sorixelle
Copy link
Member

Same as the other PR, just needs the signoffs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: docs Changes to documentation about working with the repo. NOT for changes to documentation site content! type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants