-
Notifications
You must be signed in to change notification settings - Fork 62
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
last update time at /benchmarks page #176
Comments
hey @Eomm i could work on this. ill need information on the date and where you want to add it |
Here is the script we use to download the data: I think the date is there
i'm not good at design, I would put it at the bottom of the current table, but happy to see your ideas too |
folks anybody had a chance to review the PR linked to this issue? |
I took a look at the PR. I'm not too sure if what I have found is the real problem, but.. In job But then, in job By looking at the pages of The "biggest" problem is that @simoneb if you can it is worth it, I can try to do that, and if then our pipelines works, raise a PR for |
@Gesma94 thanks for looking into it. Why don't we ask the maintainer to upgrade to the latest version of the native action instead? |
I would do that, but since I'm not sure the problem is really the one I mentioned, a new release of My idea was:
Eventually, when and if a new release will come out, we will go back to using it. What do you think? |
I think @Gesma94 found the issue and it is written in the release https://github.com/actions/download-artifact/releases/tag/v4.0.0 :
I wrote this maintenance PR #209 Here it has run successfully: #211 |
The https://fastify.dev/benchmarks page lacks the last date we updated the data.
It would be cool to add it
The text was updated successfully, but these errors were encountered: