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

Move setup instruction to top and improve Dockerfile #113

Merged
merged 3 commits into from
Jun 2, 2024

Conversation

thorian93
Copy link
Contributor

@thorian93 thorian93 commented Jun 1, 2024

It took me an embarrassing amount of time to find the setup instructions.

So I merely propose to put the setup instructions on the top of the README, like most projects do.

Due to my not being too used to GitHub, I accidentally added some changes to the Dockerfile in this PR.

Before cleaning things up, @flopp please do let me know if this is something that you would be interested in. Then I will gladly proceed and work on the container topic and polish it.

@thorian93 thorian93 changed the title Move setup instruction to top. Move setup instruction to top and improve Dockerfile Jun 2, 2024
@flopp
Copy link
Owner

flopp commented Jun 2, 2024

Thank you for improving the situation. Your changes look good, I will merge them as-is.

@flopp
Copy link
Owner

flopp commented Jun 2, 2024

If you want to continue with containers, please go ahead - I don't have much experience in this area.

@flopp flopp merged commit 1ca04e9 into flopp:main Jun 2, 2024
@thorian93
Copy link
Contributor Author

Nice, that provides me with the necessary motivation to go even a little further. Thanks!
Do you use any containerization here currently? If not, I would go ahead and clean up even further.

Love the project by the way! Exactly what I need to show off my Inline Skating. 💪 😁

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