Skip to content

Fixed CI failure caused by dependabot#47

Merged
jrsaruo merged 2 commits into
developfrom
maintenance/ci-failure-by-dependabot
May 3, 2026
Merged

Fixed CI failure caused by dependabot#47
jrsaruo merged 2 commits into
developfrom
maintenance/ci-failure-by-dependabot

Conversation

@jrsaruo

@jrsaruo jrsaruo commented May 3, 2026

Copy link
Copy Markdown
Owner

Fixed issues

  • Fixed that CodeCov upload fails when PR is created by dependabot.

Maintenance

@jrsaruo jrsaruo self-assigned this May 3, 2026
@jrsaruo jrsaruo added maintenance Maintenance such as refactoring, tests or CI dependencies Pull requests that update a dependency file labels May 3, 2026
@jrsaruo jrsaruo requested a review from Copilot May 3, 2026 15:30
@codecov

codecov Bot commented May 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.92%. Comparing base (7db5a36) to head (be83f75).
⚠️ Report is 4 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      #47   +/-   ##
========================================
  Coverage    87.92%   87.92%           
========================================
  Files           13       13           
  Lines          414      414           
========================================
  Hits           364      364           
  Misses          50       50           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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 updates the CI workflow to avoid failures when Dependabot opens pull requests, and aligns the repository checkout action with the newer major version used elsewhere in the repo.

Changes:

  • Bump actions/checkout from v5 to v6 in the test workflow.
  • Skip the Codecov upload step when the workflow is triggered by dependabot[bot].
Comments suppressed due to low confidence (1)

.github/workflows/test.yml:46

  • The Codecov step is only skipped for Dependabot, but it can still fail for other PR contexts where secrets.CODECOV_TOKEN isn’t available (e.g., PRs from forks), since fail_ci_if_error: true will make the job fail when token is empty/invalid. Consider guarding the upload step on the presence of the secret (or on github.event.pull_request.head.repo.fork == false) instead of (or in addition to) checking github.actor.
      - name: Upload coverage to Codecov
        if: ${{ github.actor != 'dependabot[bot]' }}
        uses: codecov/codecov-action@v6
        with:
          files: ${{ env.CODECOV_PATH }}
          token: ${{ secrets.CODECOV_TOKEN }}
          fail_ci_if_error: true

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jrsaruo jrsaruo force-pushed the maintenance/ci-failure-by-dependabot branch from df8fb4a to be83f75 Compare May 3, 2026 15:34
@jrsaruo jrsaruo merged commit 8871821 into develop May 3, 2026
5 checks passed
@jrsaruo jrsaruo deleted the maintenance/ci-failure-by-dependabot branch May 3, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file maintenance Maintenance such as refactoring, tests or CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants