-
Notifications
You must be signed in to change notification settings - Fork 24
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
Fix running profiler in CI on the master branch #454
base: master
Are you sure you want to change the base?
Conversation
Are you able to confirm that this would actually get the right branch name? I wonder because on this branch, the branch name evaluates to |
I can confirm it, reasons being:
|
So there are a couple of routes to running integration tests that might be affecting this value. It only runs when the branch is a PR to master, but can be triggered on a PR, or it can be triggered if something is committed to an active PR, I wonder if the branch name ends up being different for those. Seems like it could be the reason for the last failure. |
Fix #453
Profiler was not running on CI on the master branch. Fixed comparison string value in the bad line. Also, add printing out the current branch name to console for logs.
This is the current latest CI run on master showing the branch name: