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 AdonisJS #343

Merged
merged 2 commits into from
Oct 18, 2024
Merged

Add AdonisJS #343

merged 2 commits into from
Oct 18, 2024

Conversation

RomainLanz
Copy link
Contributor

Hey there! 👋🏻

This PR adds AdonisJS Http server for benchmarking purpose.

I am not quite sure about some options in the package.js file, like the extra params.

AdonisJS comes by default with middleware enabled for example.

Let me know if I need to make any modification. 👍🏻

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mcollina
Copy link
Member

mcollina commented Sep 9, 2024

Can you remove Node v14, v16 and v18 from the list of envs we support?

@RomainLanz
Copy link
Contributor Author

Can you remove Node v14, v16 and v18 from the list of envs we support?

I am not sure if you are asking me to modify something in this PR or make another PR that removes the outdated Node.js version from CI.

On a side note, if needed, I could send another PR to migrate from import assertion to import attribution.

@RomainLanz
Copy link
Contributor Author

Small bump. 👋🏻

@mcollina
Copy link
Member

Sorry for the long delay, I have a long queue of GitHub notifications to manage. CI was bumped, so you should be able to rebase now.

@RomainLanz
Copy link
Contributor Author

No worries!

PR has been rebased. 👍🏻

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mcollina mcollina merged commit 12b3a07 into fastify:master Oct 18, 2024
4 checks passed
@mcollina
Copy link
Member

finally done! Sorry for the wait!

1 similar comment
@mcollina
Copy link
Member

finally done! Sorry for the wait!

@RomainLanz RomainLanz deleted the feat/adonisjs branch October 18, 2024 18:20
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