Potential - strip lines for comparison

[jan-janssen]

Summary by CodeRabbit

• Bug Fixes
  • Enhanced LAMMPS configuration file parsing to correctly handle and recognize configuration directives with leading or trailing whitespace. This improvement ensures that critical configuration parameters such as units, atom style, and dimensionality settings are accurately identified and processed regardless of formatting inconsistencies in the input files.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The code now uses l.strip().startswith(...) instead of l.startswith(...) when matching LAMMPS configuration keywords ("units", "atom_style", "dimension"). This allows the matching logic to recognize configuration lines that have leading or trailing whitespace before the keyword.

Changes

Cohort / File(s)	Summary
Whitespace-aware LAMMPS configuration matching src/pyiron_lammps/compatibility/file.py	Modified three keyword matching checks to use .strip() before .startswith() for "units", "atom_style", and "dimension" lines, enabling recognition of indented or whitespace-prefixed configuration entries.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A rabbit hops through whitespace trim,
Where LAMMPS lines once started grim,
Now .strip() makes the matching keen,
Leading spaces—no longer unseen! 🐰✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly describes the main change: stripping whitespace from lines before comparison in LAMMPS configuration handling.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

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

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #299   +/-   ##
=======================================
  Coverage   82.33%   82.33%           
=======================================
  Files          11       11           
  Lines        1155     1155           
=======================================
  Hits          951      951           
  Misses        204      204           

☔ 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.