Skip to content

Respect custom hosts#541

Merged
jimhester merged 12 commits intor-lib:masterfrom
dagola:feat/custom_host
Jan 7, 2021
Merged

Respect custom hosts#541
jimhester merged 12 commits intor-lib:masterfrom
dagola:feat/custom_host

Conversation

@dagola
Copy link
Copy Markdown
Contributor

@dagola dagola commented Sep 21, 2020

In #448 a feature to specify a custom host URL in the Remotes field for e.g. enterprise GitHub or GitLab instances is requested.
This pull request adds a feature to specify the host URL explicitly:

<type>[@host]::<username>/<repo>[@ref]

The tests added may be not complete in the sense that I was not able to test against a custom host. Maybe anyone can provide some hints for better tests?

@dagola
Copy link
Copy Markdown
Contributor Author

dagola commented Oct 21, 2020

I also added a fix such that refs with a slash are also allowed.

@dagola
Copy link
Copy Markdown
Contributor Author

dagola commented Dec 2, 2020

@jimhester Any problems with this MR? Can I improve this MR somehow so it gets merged?

@jimhester
Copy link
Copy Markdown
Member

Can you please add a bullet to NEWS? It should briefly describe the change and end with (@yourname, #issuenumber).

@dagola
Copy link
Copy Markdown
Contributor Author

dagola commented Dec 3, 2020

@jimhester I added a bullet to NEWS but the automated checks fail because of the test internal map is current in test-bioc.R fails. Looks like there is a new bioconductor version.

@dagola
Copy link
Copy Markdown
Contributor Author

dagola commented Jan 7, 2021

@jimhester Anything else I can do?

@jimhester
Copy link
Copy Markdown
Member

Thanks a watt!

@fabiandistlerkb
Copy link
Copy Markdown

That was really helpful, since I could not find it in the docs. Would you mind adding it, or point me to the place where this is documented? Especially the @host.name part was what I needed on ghe to make this work.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants