-
Notifications
You must be signed in to change notification settings - Fork 30k
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
doc: update BUILDING.md section on running tests #40933
Conversation
LGTM except for the typo in the commit message title. :) |
Whoops! Fixed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I realize this was introduced prior to this PR, but shouldn't we be using bash
instead of text
? And shouldn't we avoid to name the python
executable (which is named python3
in many systems I believe)?
384f16f
to
82851fd
Compare
@aduh95 I believe I've addressed all your suggestions. If I've missed something, please let me know! |
442d353
to
82e292b
Compare
Landed in a5d3449 |
Refs: #40920 PR-URL: #40933 Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
Refs: #40920 PR-URL: #40933 Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
Refs: #40920 PR-URL: #40933 Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
Refs: #40920