changed btype, to btype as its causing error while running tests#5401
changed btype, to btype as its causing error while running tests#5401saiudayagiri wants to merge 2 commits into
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #5401 +/- ##
========================================
Coverage 93.85% 93.85%
========================================
Files 182 182
Lines 22509 22509
Branches 3202 3202
========================================
+ Hits 21125 21126 +1
+ Misses 922 921 -1
Partials 462 462 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
@IAlibay, could you please take a look at this? Also, I noticed that a large number of files are currently failing formatting checks. I tried running Black across the entire project locally, which resolved formatting issues in around 264 files. I have already committed those changes to a separate branch. If a repository-wide formatting update would be useful and aligns with the project's expectations, please let me know and I can open a separate PR for those changes. |
|
@saiudayagiri as far as I can see, only one file is failing the formatting check: I think this was introduced in #5372 and seems to have been fixed in #5374 already. You should be able to just merge the changes on |
As this is a small change i have not raised any issue.
Changes made in this Pull Request:
pytest.mark.parametrizefortest_VEintestsuite/MDAnalysisTests/core/test_atomgroup.py."btype,", changing it to"btype"so that the parametrized argument correctly matches the test function signature.LLM / AI generated code disclosure
LLMs or other AI-powered tools (beyond simple IDE use cases) were used in this contribution: No
PR Checklist
package/CHANGELOGfile updated?package/AUTHORS? (If it is not, add it!)Developers Certificate of Origin
I certify that I can submit this code contribution as described in the [Developer Certificate of Origin](https://developercertificate.org/), under the MDAnalysis [LICENSE](https://github.com/MDAnalysis/mdanalysis/blob/develop/LICENSE).