Skip to content
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

Provide scm credentials when fetching information about remote branches. #1135

Merged
merged 2 commits into from Apr 30, 2019

Conversation

olifly
Copy link
Contributor

@olifly olifly commented Apr 29, 2019

Fix for issue #1134

Ólafur Haukur Flygenring added 2 commits April 29, 2019 13:00
…r stream is not always returned which would trigger a NullPointerException masking the true error that occurred.

Signed-off-by: Ólafur Haukur Flygenring <olafurh@wuxinextcode.com>
before fetching (fixes issue nextflow-io#1134).

Signed-off-by: Ólafur Haukur Flygenring <olafurh@wuxinextcode.com>
@pditommaso pditommaso merged commit 808847b into nextflow-io:master Apr 30, 2019
@pditommaso
Copy link
Member

Excellent. Thanks a lot for this patch.

@olifly olifly deleted the git-fetch-fix branch April 30, 2019 12:22
sivkovic pushed a commit to sivkovic/nextflow that referenced this pull request Jun 6, 2019
…es (nextflow-io#1135)

* Switched to safe navigation operator when getting an error message (Error stream is not always
returned which would trigger an NPE masking the true error that occurred.
* Provide credentials, if they exist, to the remote git repository before fetching 
(fixes issue nextflow-io#1134).

Signed-off-by: Ólafur Haukur Flygenring <olafurh@wuxinextcode.com>
sivkovic pushed a commit to sivkovic/nextflow that referenced this pull request Jun 6, 2019
…es (nextflow-io#1135)

* Switched to safe navigation operator when getting an error message (Error stream is not always
returned which would trigger an NPE masking the true error that occurred.
* Provide credentials, if they exist, to the remote git repository before fetching 
(fixes issue nextflow-io#1134).

Signed-off-by: Ólafur Haukur Flygenring <olafurh@wuxinextcode.com>
Signed-off-by: Ivkovic <sinisa.ivkovic@gmail.com>
pditommaso pushed a commit that referenced this pull request Jul 27, 2019
…es (#1135)

* Switched to safe navigation operator when getting an error message (Error stream is not always
returned which would trigger an NPE masking the true error that occurred.
* Provide credentials, if they exist, to the remote git repository before fetching 
(fixes issue #1134).

Signed-off-by: Ólafur Haukur Flygenring <olafurh@wuxinextcode.com>
@pditommaso pditommaso added this to the v19.07.0 milestone Jul 27, 2019
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.

None yet

2 participants