Skip to content
This repository has been archived by the owner on Nov 30, 2023. It is now read-only.

Development container definition for Elm (0.19.0) #101

Merged
merged 11 commits into from
Jul 25, 2019

Conversation

xWiiLLz
Copy link
Contributor

@xWiiLLz xWiiLLz commented Jul 18, 2019

I wanted to try out Elm and had to install the binaries to my computer (bummer 馃), so I decided to make this container definition instead 馃槒.

As of now, there is not vscode debugger for Elm yet, which is why the Readme guides developers to run the cli command directly to start the samples.

This definition is based on debian and includes both the Elm (0.19.0) binaries as well as the Elm VS Code extension.

The test project files were taken directly from the official Elm Architecture Tutorial.

Copy link
Member

@Chuxel Chuxel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much for the contribution!! A few comments inline below.

containers/elm-0.19.0/.devcontainer/devcontainer.json Outdated Show resolved Hide resolved
containers/elm-0.19.0/.devcontainer/Dockerfile Outdated Show resolved Hide resolved
containers/elm-0.19.0/README.md Outdated Show resolved Hide resolved
containers/elm-0.19.0/.devcontainer/devcontainer.json Outdated Show resolved Hide resolved
@@ -0,0 +1,27 @@
Copyright (c) 2014-2016, Evan Czaplicki
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can't accept contributions using alternate licenses because of the complexity of distributing them unfortunately. Is the license required here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The test-project folder is a straight clone of the elm-architecture-tutorial repository , which requires this license to be distributed. If we can't include it, I could remove that test-project entirely and make one from scratch that's a simple Hello World?

It wouldn't be as complete, but probably good enough to make sure the dev tools are working properly.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@xWiiLLz Yeah I think that would be best given the license is required in the other example. Thanks!!

@Chuxel
Copy link
Member

Chuxel commented Jul 25, 2019

Oops! Missed you'd done some commits to address the feedback. LGTM! Merging.

@Chuxel Chuxel merged commit 87d95a2 into microsoft:master Jul 25, 2019
@xWiiLLz
Copy link
Contributor Author

xWiiLLz commented Aug 4, 2019

Oops! Missed you'd done some commits to address the feedback. LGTM! Merging.

Just noticed that you had merged the PR without me making the appropriate changes (LICENSE still there). I will create a simple hello-world sample of my own and remove the licensing & other files to comply with your distribution. Will reference this in the upcoming PR.

Sorry about that @Chuxel

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants