Skip to content

Enable Pylint check for LF line endings#5445

Open
Urvity03 wants to merge 2 commits into
MDAnalysis:developfrom
Urvity03:fix/eol-linter-check
Open

Enable Pylint check for LF line endings#5445
Urvity03 wants to merge 2 commits into
MDAnalysis:developfrom
Urvity03:fix/eol-linter-check

Conversation

@Urvity03

@Urvity03 Urvity03 commented Jul 22, 2026

Copy link
Copy Markdown

Fixes #5315

Changes made in this Pull Request:

  • Updated package/.pylintrc to set expected-line-ending-format=LF under [FORMAT].
  • Added unexpected-line-ending-format to enable= under [MESSAGES CONTROL].
  • Verified that Pylint reports unexpected-line-ending-format for files containing CRLF line endings.

LLM / AI generated code disclosure

LLMs or other AI-powered tools (beyond simple IDE use cases) were used in this contribution: yes

AI assistance was used to understand the repository structure, evaluate implementation options, review the relevant Pylint configuration and documentation, and assist in drafting explanations and the pull request description.

I reviewed the proposed approach, implemented the configuration changes, verified the behavior locally, committed the changes, and submitted this pull request.

PR Checklist

  • Issue raised/referenced?
  • Tests updated/added?
  • Documentation updated/added?
  • package/CHANGELOG file updated?
  • Is your name in package/AUTHORS? (If it is not, add it!)
  • LLM/AI disclosure was updated.

Developers Certificate of Origin

I certify that I can submit this code contribution as described in the Developer Certificate of Origin, under the MDAnalysis LICENSE.

@read-the-docs-community

read-the-docs-community Bot commented Jul 22, 2026

Copy link
Copy Markdown

Documentation build overview

📚 MDAnalysis | 🛠️ Build #33711126 | 📁 Comparing 9105f04 against latest (0803d88)

  🔍 Preview build  

3 files changed
± index.html
± documentation_pages/analysis/wbridge_analysis.html
± documentation_pages/coordinates/XDR.html

@orbeckst orbeckst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for your contribution. However, we require our standard PR header, which includes a number of attestations, including a statement on AI use.

Please restore the standard PR header and fill that one in. We will not review PRs that do not follow our standard PULL_REQUEST_TEMPLATE.

@Urvity03

Urvity03 commented Jul 22, 2026

Copy link
Copy Markdown
Author

Thank you for your contribution. However, we require our standard PR header, which includes a number of attestations, including a statement on AI use.

Please restore the standard PR header and fill that one in. We will not review PRs that do not follow our standard PULL_REQUEST_TEMPLATE.

Thanks for the feedback! I've updated the PR to use the project's standard pull request template, completed the required AI disclosure, and added my name to package/AUTHORS as requested by the template. Please let me know if there's anything else I should update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Modify linter config to check end-of-line characters

2 participants