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 doc for job runtime #45

Merged
merged 5 commits into from
Dec 12, 2022
Merged

Add doc for job runtime #45

merged 5 commits into from
Dec 12, 2022

Conversation

tuliren
Copy link
Member

@tuliren tuliren commented Dec 11, 2022

No description provided.

@stoat-app
Copy link

stoat-app bot commented Dec 11, 2022

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

Static Hosting

Name Link Commit Status
Documentation Visit ae6b21c

Job Runtime

debug

@@ -33,14 +37,15 @@ jobs:
uses: actions/cache@v3
with:
path: |
~/.npm
Copy link
Member Author

Choose a reason for hiding this comment

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

This line shortens the installation time from 20s to 4s. It is not shown in the runtime chart because we currently always use the first run for each commit. This is fixed in https://github.com/stoat-dev/stoat/pull/92. After that PR is merged, the shortened runtime should show up in the chart.

@tuliren tuliren requested a review from jrhizor December 11, 2022 05:38
enabled: true
tasks:
# Add a task with any arbitrary name.
job-runtime:
Copy link
Contributor

Choose a reason for hiding this comment

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

I would call this something different like build-time-history or something just so it's clear which key represents the plugin itself.

Copy link
Member Author

Choose a reason for hiding this comment

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

Will do.

@tuliren tuliren merged commit de01d9a into main Dec 12, 2022
@tuliren tuliren deleted the liren/job-runtime-doc branch December 12, 2022 00:40
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