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

Add Docker functionality for Solder #715

Merged
merged 4 commits into from Aug 2, 2023
Merged

Conversation

Dark-Arcana
Copy link
Contributor

This PR adds Docker functionality to TechnicSolder. Docker gives the solder user the ability to quickly setup an instance without needing to worry about what software runs on their system. As long as they can install Docker / Docker Compose, this will run. I was setting it up for a separate project (Endermedia solder) and worked a little with Pyker. There was another open PR, but it doesn't really work and doesn't have any contextual information. So with some help, I set this up and tested for reproducibility by following the steps outlined in the readme.

This PR makes zero changes to the app or php. It just adds some docker stuff. It won't effect solder if a person chooses to not use it. But if they do choose to use it, it can save them some time in getting things setup.

If there was any sort of setup that I missed along the way, please let me know. For example, I didn't know Redis was a thing before starting this xD. I'm not sure if it's setup correctly but everything appeared to work with it.

Copy link
Member

@Pyker Pyker left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏼

@Pyker Pyker changed the title Add Docker functionality for Solder. Add Docker functionality for Solder Aug 2, 2023
@Pyker Pyker merged commit 606fec2 into TechnicPack:dev Aug 2, 2023
@Pyker
Copy link
Member

Pyker commented Aug 2, 2023

Thanks!

@Pyker Pyker mentioned this pull request Aug 2, 2023
2 tasks
@Pyker Pyker mentioned this pull request Oct 17, 2023
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.

None yet

2 participants