Looks like you hit rate limits, can you continue and answer my question above?
What are the performance characteristics of this pattern? Specifically in two cases:
@mahseiah_ai Can you pick this up and implement it? Let's also bump the minor version number for the package. When the PR is ready I'll review it. Once its approved and we merge, we should tag the…
Regarding the open questions:
- We should mention
npm publish, but as part of the current manual release process. This process is (in brief):- Ensure the library builds and passes test…
If we switch to the bun runner as I asked in my previous review comment, we will not need to install bun. It comes packaged in that runner.
Overall it looks great! Just one change to the Actions workflow runner config.
Can we use the bun runner? It is more lightweight than ubuntu-latest and should have everything we need. bun maps to the docker://oven/bun:latest image.
@mahseiah_ai I agree. I think there is only one open questions: Which test runner should we use? This is a very lightweight library. I am leaning towards using bun test to avoid bringing in any…