Skip to content

WEB-146: Integrate Codecov coverage upload - #3743

Open
SaaiAravindhRaja wants to merge 1 commit into
openMF:devfrom
SaaiAravindhRaja:WEB-146-integrate-codecov
Open

WEB-146: Integrate Codecov coverage upload#3743
SaaiAravindhRaja wants to merge 1 commit into
openMF:devfrom
SaaiAravindhRaja:WEB-146-integrate-codecov

Conversation

@SaaiAravindhRaja

@SaaiAravindhRaja SaaiAravindhRaja commented Jul 21, 2026

Copy link
Copy Markdown
Member

Description

Integrates Codecov upload into the existing build workflow. The workflow now runs the Jest CI suite with lcov and text-summary coverage reporters, then uploads coverage/lcov.info using codecov/codecov-action@v5.

The coverage and Codecov steps are non-blocking, so existing Jest failures or upload issues do not fail the build workflow. This keeps the current build behavior intact while still publishing coverage when lcov.info is generated.

Related issues and discussion

Jira: https://mifosforge.jira.com/browse/WEB-146

Screenshots, if any

No UI changes.

Checklist

  • If you have multiple commits please combine them into one commit by squashing them.
  • Read and understood the contribution guidelines at web-app/.github/CONTRIBUTING.md.

Testing

  • ruby -e 'require "yaml"; YAML.load_file(".github/workflows/build.yml"); puts "yaml_ok"'
  • actionlint -ignore 'input "GITHUB_TOKEN" is not defined' -ignore 'input "PRESERVE" is not defined' .github/workflows/build.yml
  • ./node_modules/.bin/prettier .github/workflows/build.yml --check
  • git diff --check origin/dev...HEAD
  • gitleaks detect --no-banner --redact --source . --log-opts origin/dev..HEAD
  • PR CI confirmed Codecov found coverage/lcov.info and queued the upload for processing.

@SaaiAravindhRaja
SaaiAravindhRaja requested a review from a team July 21, 2026 11:15
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • .github/workflows/build.yml is excluded by !.github/** and included by **/*

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: e4cd5976-592c-4454-99a9-fbc9d046b0cb

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Note

.coderabbit.yaml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings (1)
Validation error: Unrecognized key: "pre_merge_checks"
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov-commenter

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@SaaiAravindhRaja
SaaiAravindhRaja force-pushed the WEB-146-integrate-codecov branch from fdd98d9 to 7b6c969 Compare July 21, 2026 11:29
@IOhacker

Copy link
Copy Markdown
Contributor

@SaaiAravindhRaja please do a rebase

@IOhacker IOhacker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@SaaiAravindhRaja @DavidH-1 Mifos has CodeCov accounts and the tokenis configured in the Github repository?

@SaaiAravindhRaja
SaaiAravindhRaja force-pushed the WEB-146-integrate-codecov branch from d1d49bf to e110876 Compare July 23, 2026 11:26
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.

3 participants