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

Added SonarQube support #1140

Merged
merged 2 commits into from
Feb 29, 2016
Merged

Added SonarQube support #1140

merged 2 commits into from
Feb 29, 2016

Conversation

Slesa
Copy link
Contributor

@Slesa Slesa commented Feb 16, 2016

and Centigrade as user

@forki
Copy link
Member

forki commented Feb 29, 2016

awesome thx

@philipooo
Copy link
Contributor

Maybe you could also add the optional /d:sonar.branch parameter.
In my case thats useful because analysis of feature branches will be seperated from the masters analysis if I populate it with e.g. git`s branch name.

@forki forki merged commit acc2d9e into fsprojects:master Feb 29, 2016
@Slesa
Copy link
Contributor Author

Slesa commented Mar 3, 2016

Should this possibly be a list of parameters to add with the /d switch or a simple string?

@philipooo
Copy link
Contributor

A list of parameters to add with the /d switch would be also very nice and would bring lots of flexibility.

@Slesa
Copy link
Contributor Author

Slesa commented Mar 9, 2016

Maybe you could test #1172, please?

@Slesa Slesa deleted the develop branch March 23, 2016 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Slesa @forki @philipooo