-
Notifications
You must be signed in to change notification settings - Fork 10
Implement results tree-report command
#213
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
Conversation
30de9d0 to
f24f752
Compare
|
Sample command output without |
|
Sample output with |
f24f752 to
9bfd141
Compare
|
Thanks for this, Jeny!
|
312c653 to
24aa97a
Compare
Add `dashboard_fetch_tree_report` to get tree report using dashboard API. Signed-off-by: Jeny Sadadia <jeny.sadadia@collabora.com>
Fetch tree report using the dashboard API endpoint and display it. The report will consist of builds, boots, tests summary along with regression, unstable tests, and build issues information. Signed-off-by: Jeny Sadadia <jeny.sadadia@collabora.com>
24aa97a to
8bfa486
Compare
|
Updated the command to include architecture/compiler information based on kernelci/dashboard#1453. |
|
looks good for me |
|
what is history: > > > representing?
but yes having documentation would be nice anyway |
|
@padovan I would prefer if we use the Unicode version of each emoji symbol, |
That is different test status over different commits for the same test. |
As I said in the same comment, I could not see emoji from my Gentoo laptopt as not every os have emoji installed by default for example I see this code as this: def status_mapping(status):
if status == "PASS":
return ""
if status == "FAIL":
return ""
if status in ["ERROR", "SKIP", "MISS", "DONE", "NULL", None]:
return ""I think we should made emoji optional in the future |
Get tree report.