Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@davorrunje
Copy link
Collaborator

@davorrunje davorrunje commented Jan 22, 2025

Why are these changes needed?

Related issue number

Closes #603

Checks

hasChanges: ${{ steps.filter.outputs.autogen == 'true' || steps.filter.outputs.test == 'true' || steps.filter.outputs.workflows == 'true' || steps.filter.outputs.setup == 'true' }}
steps:
- uses: actions/checkout@v4
- uses: dorny/paths-filter@v2

Check warning

Code scanning / CodeQL

Unpinned tag for a non-immutable Action in workflow Medium test

Unpinned 3rd party Action 'Tests with optional dependencies' step
Uses Step: filter
uses 'dorny/paths-filter' with ref 'v2', not a pinned commit hash
bash scripts/test.sh test/test_import.py
bash scripts/test.sh -m ${{ matrix.optional-dependencies }}
- name: Upload coverage to Codecov
uses: codecov/codecov-action@v3

Check warning

Code scanning / CodeQL

Unpinned tag for a non-immutable Action in workflow Medium test

Unpinned 3rd party Action 'Tests with optional dependencies' step
Uses Step
uses 'codecov/codecov-action' with ref 'v3', not a pinned commit hash
@davorrunje davorrunje added this pull request to the merge queue Jan 27, 2025
Merged via the queue into main with commit db66137 Jan 27, 2025
292 of 299 checks passed
@davorrunje davorrunje deleted the add-ci-tests-for-optional-deps branch January 27, 2025 16:00
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.

Create new CI tests for optional dependencies

6 participants