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 code coverage report to web/crux scope #836

Open
polaroi8d opened this issue Sep 28, 2023 · 6 comments
Open

Add code coverage report to web/crux scope #836

polaroi8d opened this issue Sep 28, 2023 · 6 comments
Assignees
Labels
issue:hacktoberfest Issue marked as suitable for Hacktoberfest issue:help wanted Extra attention is needed issue:improvement Issue that makes something better. lang:javascript lang:typescript severity:medium

Comments

@polaroi8d
Copy link
Collaborator

Currently, we use CodeCov to report Golang test coverage in our repository, but we're missing coverage information for the web scope. As a first step, we aim to add the web/crux scope to the coverage report. We use Jest in this scope to test mapper functions. Therefore, the task involves investigating and implementing a solution to include these tests in the coverage report.

\Additionally, we modify the GitHub Actions workflow to check the new coverage and automatically comment the new report on pull requests.

Please note that we are open to discussing and assisting with any related issues, even if someone who isn't one of the main contributors wishes to edit or add items to the CodeCov function. Happy to discuss this topic in detail.

@polaroi8d polaroi8d added issue:help wanted Extra attention is needed issue:improvement Issue that makes something better. issue:hacktoberfest Issue marked as suitable for Hacktoberfest lang:typescript severity:medium lang:javascript labels Sep 28, 2023
@polaroi8d polaroi8d pinned this issue Sep 28, 2023
@sahil9001
Copy link

sahil9001 commented Oct 2, 2023

Hey @polaroi8d , can you please assign me this issue? Also a starting point would be helpful for this issue

@polaroi8d
Copy link
Collaborator Author

Hi @sahil9001, I've been assigned to this task, but as it's currently written, the only starting point is that we already have Codecov, a code coverage reporting tool, and we are using Jest to write tests in the codebase. So, the task is to investigate and implement a proper solution for adding the project scope to the Codecov report. We are open to discussing further moves ✋🏻

@polaroi8d polaroi8d assigned polaroi8d and sahil9001 and unassigned polaroi8d Oct 4, 2023
@sahil9001
Copy link

Sure, let me know the next steps, from what I understood, we need to include project scope in codecov which will be covered by jest during testing, right?

@polaroi8d
Copy link
Collaborator Author

Yes @sahil9001

@polaroi8d polaroi8d unpinned this issue Dec 4, 2023
@Souravpakhira
Copy link
Contributor

Hey hawkeye here from discord please assign this issue to me

@m8vago m8vago assigned Souravpakhira and unassigned sahil9001 Dec 20, 2023
@Souravpakhira
Copy link
Contributor

#898 sorry for the delay

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue:hacktoberfest Issue marked as suitable for Hacktoberfest issue:help wanted Extra attention is needed issue:improvement Issue that makes something better. lang:javascript lang:typescript severity:medium
Projects
None yet
Development

No branches or pull requests

3 participants