-
Notifications
You must be signed in to change notification settings - Fork 13
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
expand build and running instructions for different platforms #13
Comments
Mac build/run instruction could be a good improvement, right? |
Ok, I was able to build, and run it with the scripts, but I have no idea how to debug it for example :) |
I tried to create a PR with improved instructions but I was not able to do it :(
|
@czp13 Thanks for your interest in contributing. Feel free to fork the repo and submit a PR that way, we can than review. Thanks! https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request |
@czp13 Hey Peter it was really great meeting you at Devoxx last week and I'm glad to see you are still interested in Trinity. |
Hey @samypr100 and @Birdasaur, (I will use GitHub names as you did not share other info, so I will stick to it ;)) It was nice to meet with both of you as well last week at the conference. Also, I think let us drop the LinkedIn conversation, easier to do here, as it is more in context. Project:
No worries I will play around (maybe my JDK version, etc) when I have time, not working to Vaadin. |
Also Forking works, I was just in a lucky situation that I was a contributor to opensource projects from the inside so was easy to review, create PR-s, etc, but I will see the other side now :D |
Nice work! |
Current description is very minimal.
The text was updated successfully, but these errors were encountered: