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

onboard to Codecov for code coverage #1339

Closed
tianleh opened this issue Mar 10, 2022 · 2 comments · Fixed by #1478
Closed

onboard to Codecov for code coverage #1339

tianleh opened this issue Mar 10, 2022 · 2 comments · Fixed by #1478
Assignees
@tmarkley
Labels
enhancement New feature or request

Comments

@tianleh
Copy link
Member

tianleh commented Mar 10, 2022 •
edited by ahopp
Loading

Is your feature request related to a problem? Please describe.

There is an project initiative for code coverage. See examples
https://github.com/opensearch-project/opensearch-build/blob/main/.codecov.yml
https://github.com/opensearch-project/security-dashboards-plugin/blob/main/codecov.yml

Describe the solution you'd like

onboard to Codecov

Describe alternatives you've considered

Additional context
@tianleh tianleh added the enhancement New feature or request label Mar 10, 2022
@kavilla
Copy link
Member

kavilla commented Mar 10, 2022

Linking similar issue: #1035

@kavilla
Copy link
Member

kavilla commented Mar 17, 2022

I believe @ananzh was looking into this?

@tmarkley tmarkley self-assigned this Apr 18, 2022
tmarkley pushed a commit to tmarkley/OpenSearch-Dashboards that referenced this issue Apr 19, 2022
Adds code coverage reporting
878139f 
* Renames build and test workflow to eliminate ambiguity.
* Adds coverage generation for jest and mocha tests.
* Expands coverage collection for jest tests.
* Utilizes `lcov` reporter as well as text summary reporter.

Resolves opensearch-project#1339
Prerequisite for opensearch-project#1035

Signed-off-by: Tommy Markley <[email protected]>
@tmarkley tmarkley mentioned this issue Apr 19, 2022
Merged
1 task
tmarkley pushed a commit that referenced this issue Apr 19, 2022
Adds code coverage reporting (#1478)
1caf907 
* Renames build and test workflow to eliminate ambiguity.
* Adds coverage generation for jest and mocha tests.
* Expands coverage collection for jest tests.
* Utilizes `lcov` reporter as well as text summary reporter.

Resolves #1339
Prerequisite for #1035

Signed-off-by: Tommy Markley <[email protected]>
pjfitzgibbons pushed a commit to pjfitzgibbons/OpenSearch-Dashboards that referenced this issue Aug 10, 2022
Adds code coverage reporting (opensearch-project#1478)
6f8531d 
* Renames build and test workflow to eliminate ambiguity.
* Adds coverage generation for jest and mocha tests.
* Expands coverage collection for jest tests.
* Utilizes `lcov` reporter as well as text summary reporter.

Resolves opensearch-project#1339
Prerequisite for opensearch-project#1035

Signed-off-by: Tommy Markley <[email protected]>
opensearch-trigger-bot bot pushed a commit that referenced this issue Sep 28, 2022
@github-actions
Adds code coverage reporting (#1478)
ce64f53 
* Renames build and test workflow to eliminate ambiguity.
* Adds coverage generation for jest and mocha tests.
* Expands coverage collection for jest tests.
* Utilizes `lcov` reporter as well as text summary reporter.

Resolves #1339
Prerequisite for #1035

Signed-off-by: Tommy Markley <[email protected]>
(cherry picked from commit 1caf907)
abbyhu2000 added a commit that referenced this issue Oct 4, 2022
@opensearch-trigger-bot @abbyhu2000
Adds code coverage reporting (#1478) (#2444)
b40825c 
* Renames build and test workflow to eliminate ambiguity.
* Adds coverage generation for jest and mocha tests.
* Expands coverage collection for jest tests.
* Utilizes `lcov` reporter as well as text summary reporter.

Resolves #1339
Prerequisite for #1035

Signed-off-by: Tommy Markley <[email protected]>
(cherry picked from commit 1caf907)

Co-authored-by: Tommy Markley <[email protected]>
Co-authored-by: abbyhu2000 <[email protected]>
manasvinibs pushed a commit to manasvinibs/OpenSearch-Dashboards that referenced this issue Jan 19, 2023
@manasvinibs
Adds code coverage reporting (opensearch-project#1478)
4a31624 
* Renames build and test workflow to eliminate ambiguity.
* Adds coverage generation for jest and mocha tests.
* Expands coverage collection for jest tests.
* Utilizes `lcov` reporter as well as text summary reporter.

Resolves opensearch-project#1339
Prerequisite for opensearch-project#1035

Signed-off-by: Tommy Markley <[email protected]>
(cherry picked from commit 1caf907)
@manasvinibs manasvinibs mentioned this issue Jan 19, 2023
Closed
8 tasks
manasvinibs pushed a commit to manasvinibs/OpenSearch-Dashboards that referenced this issue Jan 19, 2023
@manasvinibs
Adds code coverage reporting (opensearch-project#1478)
6973f50 
* Renames build and test workflow to eliminate ambiguity.
* Adds coverage generation for jest and mocha tests.
* Expands coverage collection for jest tests.
* Utilizes `lcov` reporter as well as text summary reporter.

Resolves opensearch-project#1339
Prerequisite for opensearch-project#1035

Signed-off-by: Tommy Markley <[email protected]>
(cherry picked from commit 1caf907)
manasvinibs pushed a commit to manasvinibs/OpenSearch-Dashboards that referenced this issue Jan 20, 2023
@manasvinibs
Adds code coverage reporting (opensearch-project#1478)
68f1bfb 
* Renames build and test workflow to eliminate ambiguity.
* Adds coverage generation for jest and mocha tests.
* Expands coverage collection for jest tests.
* Utilizes `lcov` reporter as well as text summary reporter.

Resolves opensearch-project#1339
Prerequisite for opensearch-project#1035

Signed-off-by: Tommy Markley <[email protected]>
(cherry picked from commit 1caf907)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tmarkley tmarkley

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adds code coverage reporting tmarkley/OpenSearch-Dashboards
3 participants
@tmarkley @kavilla @tianleh