Skip to content
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

add a full README #87

Merged
merged 7 commits into from
Dec 21, 2022
Merged

add a full README #87

merged 7 commits into from
Dec 21, 2022

Conversation

jrhizor
Copy link
Contributor

@jrhizor jrhizor commented Dec 21, 2022

No description provided.

Copy link
Member

@tuliren tuliren left a comment

Choose a reason for hiding this comment

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

This looks beautiful!

</p>

<p align="center">
<img src="https://stoat-dev--static.stoat.page/screenshot-python.png" alt="screenshot" width="600">
Copy link
Member

Choose a reason for hiding this comment

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

Nitpick.

This runtime chart only has two data points. It is not immediately clear what it is about, mainly because on the x-axis the two commits are at the ends, and not easy to see.

What about using the chart from #77 (comment)

The static hosting section can remain the same, because the current screenshot has more entries.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll leave as is for now but then add a better screenshot once we have the default branch's data prefixed to it. I think that will be the most realistic: one or two branch dots after the 10 most recent commits.

@stoat-dev stoat-dev deleted a comment from vercel bot Dec 21, 2022
@stoat-app
Copy link

stoat-app bot commented Dec 21, 2022

Easy and customizable dashboards for your build system. Learn more about Stoat ↗︎

Static Hosting

Name Link Commit Status
Documentation Visit a8ee3e1

Job Runtime

debug

@jrhizor jrhizor merged commit f78e4c0 into main Dec 21, 2022
@jrhizor jrhizor deleted the jrhizor/real-readme branch December 21, 2022 04:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants