Skip to content

Temporal fix for build testing dependencies - #404

Merged
Axel Suárez (axelsrz) merged 1 commit into
mainfrom
users/axsuarez/build-fix
May 22, 2026
Merged

Temporal fix for build testing dependencies#404
Axel Suárez (axelsrz) merged 1 commit into
mainfrom
users/axsuarez/build-fix

Conversation

@axelsrz

Copy link
Copy Markdown
Member

This pull request makes a minor update to the test commands in the CI workflows to suppress a specific deprecation warning during pytest runs.

  • CI/CD workflow updates:
    • Both .azdo/ci-pr.yaml and .github/workflows/python-package.yml have been updated to run pytest with a warning filter to ignore the "SelectableGroups dict interface is deprecated" DeprecationWarning. [1] [2]

Copilot AI review requested due to automatic review settings May 21, 2026 23:51
@axelsrz
Axel Suárez (axelsrz) marked this pull request as ready for review May 21, 2026 23:51
@axelsrz
Axel Suárez (axelsrz) requested a review from a team as a code owner May 21, 2026 23:51

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

Updates CI test commands to suppress a specific DeprecationWarning during pytest runs, keeping builds green while an upstream dependency emits the warning.

Changes:

  • Add a pytest -W ignore:...:DeprecationWarning filter to the GitHub Actions workflow test step.
  • Add the same warning filter to the Azure DevOps PR pipeline test step.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/python-package.yml Runs pytest with a DeprecationWarning ignore filter for the SelectableGroups deprecation message.
.azdo/ci-pr.yaml Runs pytest with the same warning filter in the AzDO CI PR pipeline.

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

Comment thread .github/workflows/python-package.yml
Comment thread .azdo/ci-pr.yaml
@axelsrz
Axel Suárez (axelsrz) merged commit 62bd709 into main May 22, 2026
12 checks passed
@axelsrz
Axel Suárez (axelsrz) deleted the users/axsuarez/build-fix branch May 22, 2026 00:11
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