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

Added Dockerfile and nginx.conf, updated README.md #53

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

Conversation

FiveVoltLogic
Copy link

I'd like to start with: this is my first contribution to a public repo. If anything is wrong, let me know and I'll do my best to resolve asap.

I run debian linux and haven't been able to find a way to run PKHEX natively on the desktop without some serious hacks. This seems like the best option for me at the moment and a nicer interface than using the command line for everything.

I have included a Dockerfile, nginx.conf and updated the README.md with some instructions for anyone wanting to deploy with docker.

I commented out the RUN dotnet restore in the Dockerfile as it was causing issues, I forgot to remove it entirely.

@codemonkey85 codemonkey85 self-assigned this Oct 17, 2024
@codemonkey85
Copy link
Owner

@FiveVoltLogic I'll be honest, I have very little experience with Docker. If you had to comment out RUN dotnet restore, are you sure the file is correct? Is someone who doesn't have a local install of .NET going to have an issue with this?

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