Skip to content

ci: zizmor fixes - #2277

Merged
RobPasMue merged 26 commits into
mainfrom
ci/zizmor
Oct 2, 2025
Merged

RobPasMue merged 26 commits into
mainfrom
ci/zizmor

Conversation

@RobPasMue

Copy link
Copy Markdown
Member

Description

Implementing CI best practices.

Issue linked

Closes #2276

@RobPasMue
RobPasMue requested a review from a team as a code owner October 1, 2025 08:34
@github-actions github-actions Bot added the maintenance Package and maintenance related label Oct 1, 2025
@RobPasMue RobPasMue self-assigned this Oct 1, 2025
@codecov

codecov Bot commented Oct 1, 2025 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.38%. Comparing base (cd1197f) to head (49c2f4d).
⚠️ Report is 2 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (cd1197f) and HEAD (49c2f4d). Click for more details.

HEAD has 4 uploads less than BASE
Flag BASE (cd1197f) HEAD (49c2f4d)
6 2
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2277      +/-   ##
==========================================
- Coverage   94.52%   85.38%   -9.14%     
==========================================
  Files         169      157      -12     
  Lines       11034    10862     -172     
==========================================
- Hits        10430     9275    -1155     
- Misses        604     1587     +983     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@RobPasMue
RobPasMue requested a review from moe-ad October 1, 2025 10:44
@RobPasMue

Copy link
Copy Markdown
Member Author

@moe-ad I'd appreciate it if you could have a look at it

moe-ad
moe-ad previously approved these changes Oct 1, 2025

@moe-ad moe-ad left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM. Quite a big change.

@RobPasMue
RobPasMue merged commit b94575b into main Oct 2, 2025
23 of 26 checks passed
@RobPasMue
RobPasMue deleted the ci/zizmor branch October 2, 2025 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Package and maintenance related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MAINTENANCE] Missing or outdated ansys/actions/check-actions-security action in ansys/pyansys-geometry

3 participants