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

Can't set gitlab repository in elm-package.json #777

Closed
diugalde opened this Issue Dec 8, 2016 · 3 comments

Comments

Projects
None yet
3 participants
@diugalde

diugalde commented Dec 8, 2016

My elm-package.json looks like:

{
    ....
    "repository": "https://gitlab.com/org/name.git",
    ....
}

I get the following message:

Error: The description in elm-package.json is not valid.

Error in $: Problem with the "repository" field.

The given domain does not start with <https://github.com/>

I think is wrong to restrict the repositories to GitHub.

@process-bot

This comment has been minimized.

Show comment
Hide comment
@process-bot

process-bot Dec 8, 2016

Thanks for the issue! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

process-bot commented Dec 8, 2016

Thanks for the issue! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

@jvoigtlaender

This comment has been minimized.

Show comment
Hide comment
@jvoigtlaender

jvoigtlaender Dec 8, 2016

Contributor

This has been discussed often. And/but this here is the wrong repository for that discussion.

Contributor

jvoigtlaender commented Dec 8, 2016

This has been discussed often. And/but this here is the wrong repository for that discussion.

@jvoigtlaender

This comment has been minimized.

Show comment
Hide comment
@jvoigtlaender

jvoigtlaender Dec 9, 2016

Contributor

For reference: elm-lang/elm-package#189.

Contributor

jvoigtlaender commented Dec 9, 2016

For reference: elm-lang/elm-package#189.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment