ErnestOrt / Trampoline

Admin Spring Boot Locally
http://ernestort.github.io/Trampoline/
Apache License 2.0
356 stars 82 forks source link

Add SSH Cloning #68

Closed snowe2010 closed 6 years ago

snowe2010 commented 6 years ago

Handle ssh credentials properly.

chrome_2018-07-21_19-47-03 chrome_2018-07-21_21-28-19

ErnestOrt commented 6 years ago

Thanks @snowe2010 for time invested here!!

We will review it and we are going to tested before releasing it.

We are going to apply necessary chenges in order to keep code homogenized, it could take few days.

Cheers and do not forget to provide an star!

snowe2010 commented 6 years ago

@ErnestOrt no problem. If you have specific code styles you'd like applied or something rearchitected let me know and I can take a stab at it as well. It seems like it could be better organized, but I wasn't sure how you'd want it done, so I did it the easiest way possible.

snowe2010 commented 6 years ago

Hi @ErnestOrt, is there anything I need to do to help get this PR merged? And also what is the best place to have discussion about the direction of the application? Some of my team members have some suggestions that I'd wonder if you'd like to hear.

ErnestOrt commented 6 years ago

Hi @snowe2010 ,

I was planning to review it during this weekend.

Regarding suggestion, we would be really glad to hear, and even implementing, them. I would recommend to open an issue for each suggestion so we can comment them separately. Moreover, they will be labelled as improvement.

Thanks in advance for your time invested on Trampoline!

snowe2010 commented 6 years ago

👏