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 link to benchmarks in GitHub readme #1063

Merged
merged 2 commits into from
Mar 12, 2024
Merged

Conversation

fredrik-bakke
Copy link
Collaborator

No description provided.

@fredrik-bakke fredrik-bakke changed the title Add link to benchmarks in GitHub README Add link to benchmarks in GitHub readme Mar 10, 2024
@fredrik-bakke fredrik-bakke added the documentation Improvements or additions to documentation label Mar 10, 2024
@VojtechStep
Copy link
Collaborator

I think I'd rather advertise a public link once the website is hosted somewhere else than on my personal Netlify account, but migrating it isn't very high on my list of priorities.

@fredrik-bakke
Copy link
Collaborator Author

fredrik-bakke commented Mar 10, 2024

It is too bad if we end up never sharing our benchmarking data simply because it is hosted in the wrong place. Therefore I suggest we share the link to the current hosting location until it is changed.

@VojtechStep
Copy link
Collaborator

I don't have a strong opinion here, but a part of why migrating isn't a high priority for me is that I don't feel the need to show this data publicly. But if you do, then I guess feel free to merge this.

@VojtechStep VojtechStep enabled auto-merge (squash) March 12, 2024 14:42
@VojtechStep VojtechStep merged commit 73b3af5 into master Mar 12, 2024
4 checks passed
@VojtechStep VojtechStep deleted the fredrik-bakke-patch-1 branch March 12, 2024 14:57
fredrik-bakke added a commit to fredrik-bakke/agda-unimath that referenced this pull request Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants