Skip to content

Revert test coverage dashboard config#252

Merged
ssagarverma merged 2 commits into
mainfrom
sv/revert-coverage-dashboard-config
Jul 2, 2026
Merged

Revert test coverage dashboard config#252
ssagarverma merged 2 commits into
mainfrom
sv/revert-coverage-dashboard-config

Conversation

@ssagarverma

Copy link
Copy Markdown
Contributor

🛠️ Description

🔗 External Links

👍 Definition of Done

  • New functionality works?
  • Tests added?

🤔 Can be merged upon approval?

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.

  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.

  • If applicable, I've documented the impact of any changes to security controls.

    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

@ssagarverma ssagarverma requested a review from a team as a code owner July 2, 2026 07:00
Copilot AI review requested due to automatic review settings July 2, 2026 07:00

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR reverts the previously-added test coverage dashboard reporting setup by removing the committed coverage report artifact and the CI automation that generated/updated it.

Changes:

  • Deletes coverage-report.json from the repository.
  • Removes the GitHub Actions workflow job that generated coverage-report.json and opened automated PRs.
  • Deletes the Python script used to generate the coverage report JSON.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
coverage-report.json Removes the committed coverage report JSON artifact.
.gitignore Updates ignore rules related to local/CI-generated artifacts.
.github/workflows/golangci.yml Removes the coverage-report update automation job (PR creation + JSON generation).
.github/scripts/generate_coverage_report.py Removes the JSON generation script used by CI.

@mohanmanikanta2299 mohanmanikanta2299 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@ssagarverma ssagarverma merged commit 4348214 into main Jul 2, 2026
6 checks passed
@ssagarverma ssagarverma deleted the sv/revert-coverage-dashboard-config branch July 2, 2026 07:20
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.

4 participants