-
Notifications
You must be signed in to change notification settings - Fork 141
feat: adding more hooks #4359
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
feat: adding more hooks #4359
Conversation
There was a problem hiding this 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 adds pre-commit hooks to enforce consistent file formatting, specifically addressing line ending issues and trailing whitespace. The changes clean up numerous data files, documentation files, and configuration files by removing extraneous blank lines at the end of files.
- Added several pre-commit hooks for file consistency checks
- Removed trailing blank lines from 225+ files across the repository
- Standardized line endings to Unix-style (LF)
Reviewed changes
Copilot reviewed 157 out of 212 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
.pre-commit-config.yaml |
Added hooks for end-of-file fixing, trailing whitespace removal, line ending standardization, and other file consistency checks |
Multiple .dat files in src/ansys/mapdl/core/examples/verif/ |
Removed trailing blank lines from verification test data files |
Multiple documentation .rst files |
Removed trailing blank lines from documentation source files |
Multiple configuration files (.yml, Dockerfile) |
Removed trailing blank lines from configuration and Docker files |
.devcontainer/ files |
Removed trailing blank lines from devcontainer configuration files |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@pyansys-ci-bot LGTM. |
pyansys-ci-bot
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4359 +/- ##
==========================================
- Coverage 91.35% 91.28% -0.07%
==========================================
Files 196 196
Lines 15977 15977
==========================================
- Hits 14596 14585 -11
- Misses 1381 1392 +11 🚀 New features to boost your workflow:
|

Description
As the title.
Initially to fix line ending issues.
Issue linked
NA
Checklist
draftif it is not ready to be reviewed yet.feat: adding new MAPDL command)