MinimalNugetServer icon indicating copy to clipboard operation
MinimalNugetServer copied to clipboard

Docker?

Open pauldotknopf opened this issue 8 years ago • 6 comments

If not, I recommend setting an image up. Maybe use Travis CI to build it.

pauldotknopf avatar Jul 14 '17 17:07 pauldotknopf

Excuse me, could you elaborate please? What you are talking about sounds is a bit vague to me ^^

TanukiSharp avatar Jul 15 '17 02:07 TanukiSharp

Sounds like a request to provide a docker image for this server - which I would also like to see.

urbanhusky avatar Sep 18 '17 11:09 urbanhusky

I've been trying to create a Dockerfile but it seems the project strangely require .NET core 1.0.5 to publish. The project shouldn't require .NET core 1.0.5 because it is not declared anywhere in the project.

After some inspection, I suspect GitCommitInfo (which required .NET core 1.0.5 to run) is the culprit. Do you have any input on this? @TanukiSharp

If I can sort this out, perhaps we can pull off a docker image for this good package.

atp-tha avatar Oct 15 '17 13:10 atp-tha

Sorry for very late reply. I'm quite busy lately and will still be until mid/end of November. Actually GitCommitInfo has bugged me several times on different projects, and most of the time I simply stopped using it, making me blind on which commit is published, but at least I can publish.

So if it bothers you, remove it from your fork. I may consider to remove it as well.

TanukiSharp avatar Oct 15 '17 13:10 TanukiSharp

If that's the case, then I'll create 2 PRs, one for removing GitCommitInfo and another one for the Dockerfile to keep the topic separated.

atp-tha avatar Oct 15 '17 16:10 atp-tha

Hi,

I forked the project, got the docker file to work and added travis ci for automated build of the CI. Will let you know how the testing of the docker image goes.

Branch https://github.com/rolfwessels/MinimalNugetServer/tree/feature/add_docker_build_scripts Travis Ci Build https://travis-ci.org/rolfwessels/MinimalNugetServer

Will do a PR back when it's all integrated.

rolfwessels avatar Dec 13 '17 11:12 rolfwessels