fastify / fastify-rate-limit

A low overhead rate limiter for your routes
MIT License
477 stars 66 forks source link

Updating for v5 #333

Closed EliphazBouye closed 8 months ago

EliphazBouye commented 10 months ago

Hi šŸ‘‹šŸ¾ I'm Eliphaz Bouye, and would like to work on this.

jsumners commented 10 months ago

I think it would be a good idea to coordinate with @gurgunday on this one. He has been doing a lot of work in this repo, and a new major should be released soon. After that, prepping a next branch should be easy to do.

gurgunday commented 10 months ago

Iā€™m down!

EliphazBouye commented 10 months ago

Ok, nice :) So @gurgunday what's the starting point?

gurgunday commented 10 months ago

Yeah so the first todo is to determine if the package should be archived -- the answer to which is a short no because we all love rate limiting ;)

Now that we crossed that off, let's move on to the second:

I will also create the next branch so that you can target it with PRs

In general it should be a fairly smooth process for this plugin

EliphazBouye commented 10 months ago

Great let's go šŸš€

EliphazBouye commented 10 months ago

@gurgunday updating is done :) . Can you create the next branch where I can target them for my PR?

jsumners commented 10 months ago

Done https://github.com/fastify/fastify-rate-limit/tree/next

gurgunday commented 8 months ago

Every task here has been completed except the workflow update (#350)

Will be tracking that from https://github.com/fastify/fastify-rate-limit/issues/340

Thanks again for the contribution