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.

Sign up for GitHub

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

Jump to bottom

Make sure that ReporterBuilder is available during batch #214

Merged
merged 1 commit into from
Mar 27, 2019
Merged

Make sure that ReporterBuilder is available during batch #214

merged 1 commit into from
Mar 27, 2019

Conversation

lukaspj
Copy link

@lukaspj lukaspj commented Jan 31, 2019

See #213

@lukaspj lukaspj mentioned this pull request Jan 31, 2019
Open
@lukaspj
Copy link
Author

lukaspj commented Jan 31, 2019

If you don't want to build it locally, here is a built version of this branch:
sonar-gitlab-plugin-4.1.0-SNAPSHOT.zip

@dmarin
Copy link

dmarin commented Feb 6, 2019

Hi @gabrie-allaigre would be great if you could merge this and release a new version. This is preventing me to upgrade to the latest sonarqube.

@DennisHartrampf
Copy link

DennisHartrampf commented Feb 6, 2019 •
edited
Loading

Today I upgraded our SonarQube to 7.6 and we are facing this issue now. I am glad, it is already fixed, and would love to see a release soon.

Edit: In the meantime we use the snapshot version you posted. Thanks for that!

@ripper2hl
Copy link

Thanks !

this a link for manual installation

https://docs.sonarqube.org/latest/setup/install-plugin/

captura de pantalla de 2019-02-13 18-01-25

merge this please

@alephtwo
Copy link

Can we get a status update on this? I'd rather not manage my own build of the plugin with this fix in, but it's a serious blocker for us upgrading Sonarqube.

@don-vip
Copy link

don-vip commented Mar 8, 2019

@gabrie-allaigre Can you please merge this fix and make a new release?

This was referenced Mar 10, 2019
Closed
Open
@alephtwo alephtwo mentioned this pull request Mar 18, 2019
Open
@gabrie-allaigre gabrie-allaigre merged commit 5d3033c into gabrie-allaigre:master Mar 27, 2019
@dmarin
Copy link

dmarin commented Mar 27, 2019

👏 👏 thanks!!

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

@freitasmurillo freitasmurillo freitasmurillo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@lukaspj @dmarin @DennisHartrampf @ripper2hl @alephtwo @don-vip @freitasmurillo @gabrie-allaigre