Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@zanderso
Copy link
Member

@zanderso zanderso commented Sep 8, 2023

No description provided.

@zanderso zanderso marked this pull request as draft September 14, 2023 15:30
zanderso added a commit that referenced this pull request Sep 15, 2023
This PR changes run_tests.py to use the python logging library to report
results instead of direct prints or writes to stdout/stderr. This change
simplifies adding a `--quiet` flag that causes the script to only
generate output if a log is emitted at WARNING or above.

Overall this is a bit of progress toward landing something like
#45595
harryterkelsen pushed a commit that referenced this pull request Oct 23, 2023
This PR changes run_tests.py to use the python logging library to report
results instead of direct prints or writes to stdout/stderr. This change
simplifies adding a `--quiet` flag that causes the script to only
generate output if a log is emitted at WARNING or above.

Overall this is a bit of progress toward landing something like
#45595
@goderbauer
Copy link
Member

(triage): @zanderso said this is still on his radar when he has time to write code...

@goderbauer
Copy link
Member

(triage): I spoke to @zanderso last week and he said the previous comment is still accurate.

@johnmccutchan
Copy link
Contributor

Could you dump the output of "gn desc" for these new rules somewhere? I'm asking because we will be using "gn desc" to enumerate all build targets and we need some way to identify the targets that use these new templates/rules.

@zanderso zanderso force-pushed the gn-run-tests-gni branch 5 times, most recently from 2b55104 to 11f8895 Compare April 9, 2024 16:25
@zanderso zanderso closed this Jul 10, 2024
@zanderso zanderso deleted the gn-run-tests-gni branch July 10, 2024 13:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants