Skip to content

Conversation

@germa89
Copy link
Collaborator

@germa89 germa89 commented Dec 12, 2025

Description

As the title.

@germa89 germa89 requested a review from a team as a code owner December 12, 2025 15:20
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the pre-commit configuration to use commit SHA references instead of version tags for better reproducibility and adds two new security-focused hooks: zizmor for GitHub workflow security analysis and bandit for Python security checks.

  • Replaced version tags with commit SHAs for all existing hooks
  • Added zizmor hook for GitHub workflow security scanning
  • Added bandit hook for Python code security analysis

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

@germa89
Copy link
Collaborator Author

germa89 commented Dec 12, 2025

@pyansys-ci-bot LGTM.

@github-actions github-actions bot added the CI/CD Related with CICD, Github Actions, etc label Dec 12, 2025
@germa89 germa89 enabled auto-merge (squash) December 12, 2025 15:21
Copy link
Contributor

@pyansys-ci-bot pyansys-ci-bot left a comment

Choose a reason for hiding this comment

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

✅ Approving this PR because germa89 said so in here 😬

LGTM

@github-actions github-actions bot added CI/CD Related with CICD, Github Actions, etc and removed CI/CD Related with CICD, Github Actions, etc labels Dec 12, 2025
@github-actions github-actions bot added CI/CD Related with CICD, Github Actions, etc and removed CI/CD Related with CICD, Github Actions, etc labels Dec 12, 2025
@codecov
Copy link

codecov bot commented Dec 12, 2025

Codecov Report

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

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4345      +/-   ##
==========================================
+ Coverage   91.27%   91.28%   +0.01%     
==========================================
  Files         196      196              
  Lines       15977    15977              
==========================================
+ Hits        14583    14585       +2     
+ Misses       1394     1392       -2     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions github-actions bot added the maintenance General maintenance of the repo (libraries, cicd, etc) label Dec 17, 2025
@germa89 germa89 changed the title ci: adding more precommit hooks fix: security zizmore issues Jan 2, 2026
@github-actions github-actions bot added the bug Issue, problem or error in PyMAPDL label Jan 2, 2026
@germa89 germa89 disabled auto-merge January 2, 2026 15:40
@germa89 germa89 changed the title fix: security zizmore issues ci: adding more precommit hooks Jan 2, 2026
@germa89 germa89 enabled auto-merge (squash) January 5, 2026 08:36
@github-actions github-actions bot added CI/CD Related with CICD, Github Actions, etc and removed CI/CD Related with CICD, Github Actions, etc dependencies maintenance General maintenance of the repo (libraries, cicd, etc) labels Jan 5, 2026
@germa89 germa89 merged commit 65a56b9 into main Jan 5, 2026
51 checks passed
@germa89 germa89 deleted the ci/adding-more-precommit-hooks branch January 5, 2026 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Issue, problem or error in PyMAPDL CI/CD Related with CICD, Github Actions, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants