Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: Deselect from project root #990

Merged
merged 1 commit into from
Jan 21, 2025
Merged

Conversation

SMoraisAnsys
Copy link
Collaborator

Previous deselection (#989) was failing because it must be specified from project root.
I tried it locally and from a folder src where I cloned pyedb, when I run pytest pyedb/tests/legacy/unit --deselect=tests/legacy/unit/test_edb.py -vvv the edb tests are correctly unselected (11 unselected tests)

For more information on that deselection requirement see #987 and #988
Note that da26422 was able to pass the file test that was previously failing.

@github-actions github-actions bot added maintenance Package and maintenance related ci/cd labels Jan 21, 2025
Copy link
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.67%. Comparing base (2e15f33) to head (8ba3dd7).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #990   +/-   ##
=======================================
  Coverage   82.67%   82.67%           
=======================================
  Files         160      160           
  Lines       21130    21130           
=======================================
  Hits        17470    17470           
  Misses       3660     3660           

@SMoraisAnsys SMoraisAnsys merged commit 726259f into main Jan 21, 2025
30 checks passed
@SMoraisAnsys SMoraisAnsys deleted the ci/deselect-from-project-root branch January 21, 2025 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/cd maintenance Package and maintenance related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants