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.

Sign up for GitHub

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

Jump to bottom

Add bun framework adapter #1072

Merged
merged 2 commits into from
Jun 5, 2024
Merged

Add bun framework adapter #1072

merged 2 commits into from
Jun 5, 2024

Conversation

SunsetTechuila
Copy link
Contributor

KnorpelSenf
KnorpelSenf approved these changes Jun 5, 2024
View reviewed changes
Copy link
Member

@KnorpelSenf KnorpelSenf left a comment

Choose a reason for hiding this comment

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

How could I forget about this? Thank you so much!

@KnorpelSenf KnorpelSenf requested a review from rojvv June 5, 2024 14:16
rojvv
rojvv reviewed Jun 5, 2024
View reviewed changes
Copy link
Member

@rojvv rojvv left a comment

Choose a reason for hiding this comment

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

Almost there. Can you also update other languages? Just search for other files named deployment-types.md.

@SunsetTechuila SunsetTechuila requested a review from rojvv June 5, 2024 20:11
@rojvv rojvv changed the title docs(guide/deployment-types): add bun framework adapter Add bun framework adapter Jun 5, 2024
rojvv
rojvv approved these changes Jun 5, 2024
View reviewed changes
Copy link
Member

@rojvv rojvv left a comment

Choose a reason for hiding this comment

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

Thank you!

@rojvv rojvv added the not translatable Translator intervention is not required. label Jun 5, 2024
@rojvv rojvv added this pull request to the merge queue Jun 5, 2024
Merged via the queue into grammyjs:main with commit 419067e Jun 5, 2024
8 of 10 checks passed
@KnorpelSenf KnorpelSenf mentioned this pull request Jun 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KnorpelSenf KnorpelSenf KnorpelSenf approved these changes

@rojvv rojvv rojvv approved these changes

Assignees
No one assigned
Labels
not translatable Translator intervention is not required.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SunsetTechuila @KnorpelSenf @rojvv