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

Bazel test reports can't be parsed #25

Open
iuliand-db opened this issue Sep 25, 2020 · 3 comments
Open

Bazel test reports can't be parsed #25

iuliand-db opened this issue Sep 25, 2020 · 3 comments

Comments

@iuliand-db
Copy link

The glob expression sets followSymLinks to false, but Bazel uses symlinks extensively, and the testlogs are placed in a symlink along the lines of bazel-testlogs -> /private/var/tmp/_bazel_iulian.dragos/8ef0a16ed66666ff343fb34c1fdc0021/execroot/intellij_with_bazel/bazel-out/darwin-fastbuild/testlogs

@jmisur
Copy link
Contributor

jmisur commented Oct 16, 2020

Hey I don't have experience with Bezel. Would you be able to do a little experiment and enable followSymlinks and see if it actually works?

@iuliand-db
Copy link
Author

Do you mean to make the change, deploy and try a fork of this repo? I couldn't find any docs on how to deploy this action (never used GitHub Actions before).

However, I wonder why followSymlinks is off by default? Seems like it could be the other way around at no cost to anyone, and the benefit that it may work for more people :)

@jmisur
Copy link
Contributor

jmisur commented Oct 16, 2020

I would be careful as symlinks can exponentionally increase the search time for files, even currently is quite slow in medium sized projects. So at best I would provide a flag to turn it on.

For testing, you can just fork the repo, create a branch with the change and use it as a gihtub action, nothing else is required.

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

No branches or pull requests

2 participants