Skip to content

fix: Consider multiple site-packages folders - #1187

Merged
klmcadams merged 4 commits into
mainfrom
fix/ideconfig
Jun 17, 2025
Merged

fix: Consider multiple site-packages folders#1187
klmcadams merged 4 commits into
mainfrom
fix/ideconfig

Conversation

@klmcadams

Copy link
Copy Markdown
Contributor

@dipinknair encountered multiple site-packages locations in a virtual environment (lib/site-packages/ and lib64/site-packages). The ansys-mechanical-ideconfig command was raising an exception when there were more than one site-packages locations, so this PR considers that possibility

@github-actions github-actions Bot added the bug Something isn't working label Jun 12, 2025
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Jun 12, 2025
@codacy-production

codacy-production Bot commented Jun 12, 2025

Copy link
Copy Markdown

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
+0.00% (target: -1.00%) 100.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (cf30cd8) 2537 2193 86.44%
Head commit (299f8a6) 2537 (+0) 2193 (+0) 86.44% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#1187) 1 1 100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@codecov

codecov Bot commented Jun 12, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.44%. Comparing base (cf30cd8) to head (299f8a6).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1187   +/-   ##
=======================================
  Coverage   86.44%   86.44%           
=======================================
  Files          39       39           
  Lines        2537     2537           
=======================================
  Hits         2193     2193           
  Misses        344      344           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@klmcadams
klmcadams requested a review from dipinknair June 12, 2025 17:46
@klmcadams
klmcadams marked this pull request as ready for review June 17, 2025 17:41
@klmcadams
klmcadams requested a review from a team as a code owner June 17, 2025 17:42
@klmcadams
klmcadams enabled auto-merge (squash) June 17, 2025 17:42
@klmcadams
klmcadams merged commit 83eecd6 into main Jun 17, 2025
@klmcadams
klmcadams deleted the fix/ideconfig branch June 17, 2025 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants