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

RFC: Removal of @microsoft/fast-benchmarks #6949

Closed
janechu opened this issue May 5, 2024 · 0 comments · Fixed by #6968
Closed

RFC: Removal of @microsoft/fast-benchmarks #6949

janechu opened this issue May 5, 2024 · 0 comments · Fixed by #6968

Comments

@janechu
Copy link
Collaborator

janechu commented May 5, 2024

Removal of @microsoft/fast-benchmarks

Reason for removal

The project Tensile is considered a good successor to the @microsoft/fast-benchmarks package and will be replacing it as a benchmarking tool for FAST packages. There will not be any work performed on this package while it is in the FAST project.

@janechu janechu mentioned this issue May 23, 2024
5 tasks
janechu added a commit that referenced this issue May 23, 2024
# Pull Request

## 📖 Description

This change removes `@microsoft/fast-benchmarks` and instead leverages the `@tensile-perf/web-components` package for local perf testing.

### 🎫 Issues

Resolves #6949

## 👩‍💻 Reviewer Notes

This makes a best effort at re-creating benchmarking done between the two methods of benchmarking. All results are still local only and are not published anywhere. That may be a future piece of work so that results may be published and used as part of a build gate workflow.

## ✅ Checklist

### General

<!--- Review the list and put an x in the boxes that apply. -->

- [x] I have included a change request file using `$ yarn change`
- [x] I have added tests for my changes.
- [x] I have tested my changes.
- [ ] I have updated the project documentation to reflect my changes.
- [x] I have read the [CONTRIBUTING](https://github.com/microsoft/fast/blob/master/CONTRIBUTING.md) documentation and followed the [standards](/docs/community/code-of-conduct/#our-standards) for this project.
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 a pull request may close this issue.

1 participant