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 tinyhttp to the TypeScript section #825

Merged
merged 5 commits into from
Dec 27, 2020
Merged

Add tinyhttp to the TypeScript section #825

merged 5 commits into from
Dec 27, 2020

Conversation

talentlessguy
Copy link
Contributor

@talentlessguy talentlessguy commented Oct 1, 2020

Please complete the following checklist if your PR is adding new link to the list:

  • I've read contributing guidelines
  • This PR does not introduce duplicates to the list
  • The link is added at the bottom of the list category
  • Suggested repository is maintained, have active community, is able to mentor new contributors and have issues with the suggested label

tinyhttp has 400+ stars, 8 open "good first issue" issues and 11 existing contributors. I think it's enough to qualify for this criteria?

  • The link I add follows the suggested pattern:
- [Repository Name](link-to-repository-label) _(label: beginner-friendly-label-in-the-repository)_ <br> Description

Example link formatting:

- [awesome-for-beginners](https://github.com/MunGell/awesome-for-beginners/labels/good-first-contribution) _(label: good-first-contribution)_ <br> A list of awesome beginners-friendly projects.

project description

tinyhttp is a modern Express-like web framework written in TypeScript and compiled to native ESM, that uses a bare minimum amount of dependencies trying to avoid legacy hell.

stats

stars - 900+
coverage - 92%+
contributors - 26+
8 open "good first issue" issues

I hope that I followed all of the requirements so looking forward for this PR to be checked :)

@MunGell MunGell added the postponed Postponed review for the duration of the freeze label Oct 1, 2020
@MunGell MunGell closed this Oct 1, 2020
@MunGell MunGell reopened this Oct 5, 2020
@MunGell MunGell removed the postponed Postponed review for the duration of the freeze label Oct 5, 2020
@MunGell
Copy link
Owner

MunGell commented Dec 25, 2020

Hi @talentlessguy

Thanks for the PR, would you be able to fix the link format?

Thanks!

@talentlessguy
Copy link
Contributor Author

@MunGell I'm not sure what's the problem is. Doesn't the link follow the format?

- [tinyhttp](https://github.com/talentlessguy/tinyhttp/issues?q=is%3Aissue+is%3Aopen+label%3A%22good+first+issue%22) _(label: good first issue)_ <br> A 0-legacy, tiny & fast web framework as a replacement of Express.
  • tinyhttp (label: good first issue)
    A 0-legacy, tiny & fast web framework as a replacement of Express.

@MunGell
Copy link
Owner

MunGell commented Dec 27, 2020

Fixed formatting in 566f19b

@MunGell MunGell merged commit d8b1d8b into MunGell:master Dec 27, 2020
@talentlessguy talentlessguy deleted the patch-1 branch December 27, 2020 17:56
@talentlessguy
Copy link
Contributor Author

thanks for merging! 😊

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