-
Notifications
You must be signed in to change notification settings - Fork 388
Unable to change paths in cobertura report #413
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
Comments
I agree, this will be nice enhancement. Instead of doing only for cobertura format, It will be good to do for all other format's except Team City. We can add one more options, |
could be similar to #408? |
I don't think so. Its different. Seems like Coverlet was using relative path earlier but @tonerdo changed it.
I think, change was done as part of #260 There is an open issue for similar request in Icov Format, #263 |
jenkins don't like absolute path, I would like to see a switch to change to relative path. |
FYI #408 (comment) |
In #614 we changed the cobertura report back to using relative paths so that the jenkins plugin is working again and the source link option is still supported. |
@daveMueller I think that we shouldn't search the "root" of a project, but the algs should simply try to find longest shared absolute(per drive on win) and add to sources, the relative parts will go inside file attribute. Report is "unrelated" to project structure and we don't know how a project will be setup, could be created at runtime, could o couldn't have a sln an so on...report wants only "locate" source files. |
@MarcoRossignoli I got it. Is it OK when I work on it? |
Yes thank's! Assigned to you, let me know if it's ok...there is no obligation but it's useful to keep track of stuff on issue list. |
Getting an issue where our application is compiled within docker and we're trying to export the cobertura xml file from it.
Since the
filename
attribute is the full path to the file, we need to do a search a replace.Adding the root of the project to
sources/source
and using relative paths forfilename
, it would be a lot easier to manager.The text was updated successfully, but these errors were encountered: