-
Notifications
You must be signed in to change notification settings - Fork 624
Added basic timing around the ghe-restore process #625
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
Added basic timing around the ghe-restore process #625
Conversation
Awesome, thank you! |
@cainejette My PR is failing on linters, which are throwing errors on code I didn't change. How do I run these linters locally against my branch? |
That shouldn't be a required step, so you should be clear from that perspective. Although we realize now that the action wasn't set up properly to trigger on PRs from forks and are addressing it in #628 If you push a commit after that goes in, will that trigger the two required build steps? |
Yes. It seems to pass the macos-latest step, but it fails on ubuntu-latest and Lint Code Base. The former error seems to be because of an outdated URI for downloading |
This is also being fixed in #628
This can be ignored |
@cainejette So can someone merge this or is there something else I need to do for this PR to go green? |
Yep! We will see it through. Thanks for contributing! |
Merged! Sorry for the delay, and thanks again for the contribution 😄 |
…orkflow-build-release-part-2 Backport 568 for 3.7: Build and Release Workflow Part 2
This PR adds basic timing output around the
ghe-restore
process, per the request from Issue #529