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

support team builds #619

Merged
merged 4 commits into from
Sep 21, 2015
Merged

support team builds #619

merged 4 commits into from
Sep 21, 2015

Conversation

Bertk
Copy link
Contributor

@Bertk Bertk commented Aug 27, 2015

There is only one log file for a VS solution for multiple projects (or modules) which might be located in the parent folder of the project root folder.

There is only one log file for a solution for multiple VS projects (or
modules) which might be located in the parent folder of the project root
folder.
@guwirth
Copy link
Collaborator

guwirth commented Aug 28, 2015

Bigger change. @jmecosta could you verify it please? Think you are also using TFS.

@jmecosta
Copy link
Member

Nahh, to expensive for us :). However I will need to use the msbuild
runner, so I might need to check that.

On Fri, Aug 28, 2015, 18:33 Günter Wirth [email protected] wrote:

Bigger change. @jmecosta https://github.com/jmecosta could you verify
it please? Think you are also using TFS.


Reply to this email directly or view it on GitHub
#619 (comment).

…9_remove_fromIOFile

remove deprecated fromIOFile
@guwirth
Copy link
Collaborator

guwirth commented Sep 16, 2015

@jmecosta & @Bertk: How should we proceed with this one?

@Bertk
Copy link
Contributor Author

Bertk commented Sep 16, 2015

Let us wait until Jorge provides the feedback. This update enables the usage of absolute paths for the report files and reuses this report for all modules. If the file does not exist (not a absolute path) the regular search rules are used to detect the report file(s).

@jmecosta
Copy link
Member

i will tomorrow do a regression test in this pr, if that is ok i think i can be merged

@guwirth
Copy link
Collaborator

guwirth commented Sep 16, 2015

@Bertk sorry for the inconvenience. But could you rebase it please...

There is only one log file for a solution for multiple VS projects (or
modules) which might be located in the parent folder of the project root
folder.
@Bertk
Copy link
Contributor Author

Bertk commented Sep 16, 2015

@guwirth I did the rebase without squash but don't know why CI fails.

@guwirth
Copy link
Collaborator

guwirth commented Sep 17, 2015

@Bertk think is a Travis problem. Sometimes one of the builds is red because of internal Travis problems. In the past I did a small change and tried again an upload. I can try to write the Travis support again. Maybe they can help this time to avoid this in future.

@guwirth guwirth added this to the M 0.9.4 milestone Sep 17, 2015
@guwirth
Copy link
Collaborator

guwirth commented Sep 18, 2015

Travis:
"Thanks for getting in touch, and sorry to hear about the troubles.

We have had a look at some of the failing jobs on your repository, and it is seems pretty hard to tell for us what might be causing this.

Have you observed any kind of pattern in these failures? It seems sometimes an expected plugin cannot be found, but sometimes it just logs an unspecific message like "cannot start SonarCube".
May I suggest to add additional debug output around these failures so it might be easier for you to spot potential causes?"

@guwirth
Copy link
Collaborator

guwirth commented Sep 20, 2015

@jmecosta did you have time for testing?

@jmecosta
Copy link
Member

Nop, I will build today and check the results tomorrow.

On Sun, Sep 20, 2015, 18:48 Günter Wirth [email protected] wrote:

@jmecosta https://github.com/jmecosta did you have time for testing?


Reply to this email directly or view it on GitHub
#619 (comment).

@jmecosta
Copy link
Member

results were ok. for me it can be merged

guwirth added a commit that referenced this pull request Sep 21, 2015
@guwirth guwirth merged commit f34cf8e into SonarOpenCommunity:master Sep 21, 2015
@guwirth
Copy link
Collaborator

guwirth commented Sep 21, 2015

@jmecosta Thx

@guwirth guwirth mentioned this pull request Oct 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants