Skip to content

display regrtests_results.json in dashboard/html page #10

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

Merged
merged 2 commits into from
May 26, 2020

Conversation

mireille-raad
Copy link
Member

@mireille-raad mireille-raad commented May 16, 2020

This is a dashboard-y page that displays the regression test results from _data/regrtests_results.json

Markdown file
pages/regression-tests-results.markdown

url.
base url + /pages/regression-tests-results.html
You can change the url of the dashboard by changing the name of that file. Same for title etc..

The page has

  • simple listing + grouping by module.
  • emojis make it easier on the eyes to spot the status without actually having to read. It is more elegant than just coloring things with red or green... or at least more fun.
  • basic filter functionality.

The layout is responsive, I tried to make the layout work as much as possible on mobile. It is decent on mobile devices, but could be better. The names of modules is long, so it takes up space on mobile devices.

Fields
I added the fields that seemed to be most frequently filled. A lot of the fields were not, so I didn't add those. Tell me if something else needs to be there.

Question
Should we add a way to navigate to different modules? does this need to be more usable?

@mireille-raad
Copy link
Member Author

screenshots

Screen Shot 2020-05-15 at 8 38 04 PM

another section

Screen Shot 2020-05-15 at 8 38 30 PM

and another

Screen Shot 2020-05-15 at 8 38 14 PM

Copy link
Member

@coolreader18 coolreader18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry this took a while to get to; this looks really great! Thanks for working on this!

@coolreader18 coolreader18 merged commit c14ba36 into RustPython:master May 26, 2020
@mireille-raad
Copy link
Member Author

Sorry this took a while to get to; this looks really great! Thanks for working on this!

sure. thanks for creating the data file and automating it.

@youknowone
Copy link
Member

I like this page a lot, thank you!

@mireille-raad
Copy link
Member Author

@youknowone 😁I am glad.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants