Skip to content

Conversation

Howl1935
Copy link
Contributor

@Howl1935 Howl1935 commented May 29, 2025

This PR was automatically generated by Turbolift to support the Code Climate Deprecation Plan. All references to Code Climate are being systematically removed from this repository.

What Was Changed

  • Test coverage setup: Removed scripts/configurations for uploading test reports to Code Climate.
  • Other dependencies: Removed direct or indirect mentions, configs, or integrations with Code Climate.

Developer Checklist

This process is automated, but manual review is required. Please ensure:

  • All Code Climate references are removed (check all modified files).
  • No critical functionality is broken (CI/CD, test reporting, etc.).
  • Alternative solutions are in place if needed (e.g., migrate to Qlty or another coverage reporter).

Generated with turbolift.

@ibcheckmarx
Copy link

ibcheckmarx commented May 29, 2025

Logo
Checkmarx One – Scan Summary & Details58529ac7-e8e9-4119-9dfc-c47278fed8e0

New Issues (1)

Checkmarx found the following issues in this Pull Request

Severity Issue Source File / Package Checkmarx Insight
LOW Unpinned Actions Full Length Commit SHA /test.yaml: 41
detailsPinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA help...
ID: X1FjB7DR%2BQkS1Eki6B1E%2BJ17%2FTQ%3D
Fixed Issues (2)

Great job! The following issues were fixed in this Pull Request

Severity Issue Source File / Package
LOW Unpinned Actions Full Length Commit SHA /test.yaml: 48
LOW Unpinned Actions Full Length Commit SHA /test.yaml: 40

env:
CC_TEST_REPORTER_ID: ${{secrets.CC_TEST_REPORTER_ID}}
with:
coverageCommand: go test ./... -coverprofile c.out
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I suppose we'll need to come up with a different action to run these tests.

Copy link
Member

Choose a reason for hiding this comment

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

ya, i think generally we can just run this command directly as the replacement.

prefix: github.com/Ibotta/sopstool

- run: go mod download && go build
- run: go test ./...
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you!

@onyxraven onyxraven merged commit 5fd74d4 into main May 29, 2025
3 checks passed
@onyxraven onyxraven deleted the CODECLIMATE_UPDATES branch May 29, 2025 19:56
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