-
Notifications
You must be signed in to change notification settings - Fork 346
Feature Request: Implement dotnet cli (net.core) #255
Comments
It should be easy as just writing a stack for it and install the .NET Extension. We did it over at gitpod.io and it works quite excellently. |
@MarcoGriep88 sounds interesting, have you taken a look at .NET Core and VS Code? |
@vince-fugnitto I'll test it out first on Gitpod.io and see if I can carry it over to here |
Thank you! I appreciate it 👍 |
Ok, i have an Dockerfile. But i am not allowed to push it. Not even to an new Branch. Can i just post it here? |
@MarcoGriep88 have you forked the repository? |
Yes, i my fork it works fine. |
Yes, i my fork its possible. |
@MarcoGriep88 it should be possible for you to open a pull request, I went to your fork and did it |
Hi @MarcoGriep88 , At a quick look it seems your image is a fork of our full image. That's not ideal since we now would have 2 images to update whenever we find a problem with the full image. Instead, you should consider one of the following:
|
Thanks, I will try this. I am not that experienced with Docker :)
Marc Dumais <[email protected]> schrieb am Mi., 30. Okt. 2019, 17:44:
… Hi @MarcoGriep88 <https://github.com/MarcoGriep88> ,
At a quick look it seems your image is a fork of our full image. That's
not ideal since we now would have 2 images to update whenever we find a
problem with the full image.
Instead, you should consider one of the following:
- extend the current full image and add just the changes you need to
cover dotnet. e.g. start your docker file with "FROM
theiaide/theia-full:latest" and just add the "delta" you need and
potentially a new entrypoint if needed.
or
- integrate your changes in the existing full image. It already
supports many languages, and we can add more
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#255?email_source=notifications&email_token=AIGNC7JQJT25DIMADNE7JO3QRG2VHA5CNFSM4JGWVR72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECU5NZY#issuecomment-548001511>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIGNC7MB3PTSHDSE4CWKO4DQRG2VHANCNFSM4JGWVR7Q>
.
|
No problem - let us know if you have questions or need help. |
I really like the Python Version for Theia. Is there any plan to create dotnet cli support for the terminal?
The text was updated successfully, but these errors were encountered: