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 performance benchmarks to alert about performance issues #215

Merged
merged 37 commits into from
Jul 26, 2024
Merged

Conversation

OptimumCode
Copy link
Collaborator

No description provided.

@krzema12
Copy link
Owner

@OptimumCode CI is green! How about we remove the check if we're inside a PR, and see if the report gets uploaded to another repo + plant a performance regression to see what happens?

@OptimumCode
Copy link
Collaborator Author

@krzema12 sure, let's do it. But I think after that it would be better to remove the published data from the repository

@OptimumCode
Copy link
Collaborator Author

Regarding performance regression: you can see here what it would look like

@OptimumCode
Copy link
Collaborator Author

@krzema12
Copy link
Owner

Works: https://krzema12.github.io/snakeyaml-engine-kmp-benchmarks/dev/bench/

Let me remove this data.

@krzema12
Copy link
Owner

@OptimumCode approved, thanks for this 🙇 🚀

@krzema12 krzema12 changed the title Add benchmark project and action to alert performance issues Add performance benchmarks to alert about performance issues Jul 26, 2024
@krzema12 krzema12 merged commit 698261d into main Jul 26, 2024
9 checks passed
@krzema12 krzema12 deleted the issue_#208 branch July 26, 2024 14:56
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