Skip to content

DEVOPS-1101: Skip codecov for bot PR dependabot#195

Merged
RomFloreani merged 2 commits into
mainfrom
DEVOPS-1101
Jun 26, 2026
Merged

DEVOPS-1101: Skip codecov for bot PR dependabot#195
RomFloreani merged 2 commits into
mainfrom
DEVOPS-1101

Conversation

@RomFloreani

@RomFloreani RomFloreani commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

DEVOPS-1101 - Skip codecov for bot PR dependabot

Copilot AI review requested due to automatic review settings June 22, 2026 15:04
@RomFloreani

Copy link
Copy Markdown
Contributor Author

Tested here with: https://github.com/MiraGeoscience/gempy-drivers/actions/runs/27961979333/job/82745655359?pr=94

It uses the git-mira --> windows latest to run it's codecov.

@github-actions github-actions Bot changed the title Devops 1101 DEVOPS-1101: Skip codecov for bot PR dependabot Jun 22, 2026

Copilot AI 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.

Pull request overview

This PR adjusts the conditional for the Codecov upload step in the reusable Python pytest workflow, aiming to skip uploads for Dependabot pull requests.

Changes:

  • Updated the Codecov step if: expression to add a Dependabot-related exclusion.

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


- name: Codecov
if: ${{ (inputs.codecov-reference-python-version) && (matrix.python-version == inputs.codecov-reference-python-version) && contains(inputs.codecov-reference-os, matrix.os) }}
if: ${{ (inputs.codecov-reference-python-version) && (matrix.python-version == inputs.codecov-reference-python-version) && contains(inputs.codecov-reference-os, matrix.os) && !(github.event_name == 'pull_request' && github.event.pull_request.user.login == 'dependabot[bot]') }}

@sebhmg sebhmg Jun 25, 2026

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.

I suppose copilot is wrong here. Since it is a reusable workflow, the event woudl eb the one of the upstream wrokflow. Right, @RomFloreani ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes correct.

@RomFloreani RomFloreani requested a review from sebhmg June 25, 2026 13:28
@RomFloreani RomFloreani merged commit 9a74334 into main Jun 26, 2026
13 checks passed
@RomFloreani RomFloreani deleted the DEVOPS-1101 branch June 26, 2026 13:18
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