Skip to content

CI: Run PyAEDT test with PYAEDT_LOCAL_SETTINGS_PATH env var - #1622

Merged
svandenb-dev merged 2 commits into
mainfrom
ci/use-pyaedt-settings-file
Nov 3, 2025
Merged

svandenb-dev merged 2 commits into
mainfrom
ci/use-pyaedt-settings-file

Conversation

@SMoraisAnsys

Copy link
Copy Markdown
Collaborator

As title says. This should be the reason why the test were failing when releasing.

Note that ansys/pyaedt#6834 should be fixing the problem even if the env variable is not defined.

@github-actions github-actions Bot added maintenance Package and maintenance related ci/cd labels Oct 30, 2025
@codecov-commenter

codecov-commenter commented Oct 30, 2025 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.89%. Comparing base (ada010d) to head (852b3f9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1622   +/-   ##
=======================================
  Coverage   70.89%   70.89%           
=======================================
  Files         226      226           
  Lines       40670    40670           
=======================================
+ Hits        28834    28835    +1     
+ Misses      11836    11835    -1     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@svandenb-dev svandenb-dev left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@svandenb-dev
svandenb-dev enabled auto-merge (squash) October 31, 2025 15:35
@svandenb-dev
svandenb-dev merged commit 179cf04 into main Nov 3, 2025
203 of 235 checks passed
@svandenb-dev
svandenb-dev deleted the ci/use-pyaedt-settings-file branch November 3, 2025 08:13
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.

4 participants