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

Test coverage requirement for template / starter repos? #4036

Closed
mikestefanello opened this issue Feb 9, 2022 · 2 comments
Closed

Test coverage requirement for template / starter repos? #4036

mikestefanello opened this issue Feb 9, 2022 · 2 comments

Comments

@mikestefanello
Copy link
Contributor

I've been wanting to submit my project, pagoda to awesome-go but the test coverage requirement is a problem for two reasons. Pagoda is a full-featured, rapid, full-stack web development starter kit (built on top of Echo and Ent). The problems are that:

  1. A very large amount of code is generated from Ent, which is an ORM. This generated code does not generate tests so the overall coverage percentage for the project is automatically dropped very low.

  2. With this project being a starter kit / base, it includes a lot of code that is purely for example (especially the routes) to highlight features, functionality and developer patterns. These don't contain tests because they are expected to be deleted by anyone using this template for an actual project of their own. I may provide some test coverage for the most functional routes at some point.

If you remove these two things, the coverage would be extremely high, but with them, it's somewhere around 25%.

Does that exclude this project from being included or is the test coverage requirement not applicable to projects like this?

@avelino
Copy link
Owner

avelino commented Feb 9, 2022

@mikestefanello we have a category especially for this type of project.
https://github.com/avelino/awesome-go#project-layout

Even though there is an automated check process, it was developed to help awesome-go maintainers, not for inclusion or exclusion of projects (proposed inclusion via pull request).

I quickly looked at the pagoda project and liked the documentation contained in the README, you really took care with the people that are going to use the project layout, 🎉 congratulations on your work.

I would have no problem with pagoda joining awesome-go, but we need to see what other maintainers think.
Open a PR proposing to include pagoda in awesome-go (in the category mentioned above) and link this issue there.

@mikestefanello mikestefanello mentioned this issue Feb 10, 2022
7 tasks
@mikestefanello
Copy link
Contributor Author

Hey @avelino - thanks for the help, great feedback and consideration to have my project included. I opened a PR: #4038

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants