mdanalysis icon indicating copy to clipboard operation
mdanalysis copied to clipboard

doc: Add missing contributors for v2.10.0 (closes #5136)

Open Aryaman-Chaudhri opened this issue 2 months ago • 2 comments

Fixes #5136

Changes made in this Pull Request:

  • Added 5 missing contributors from v2.10.0 to the CHANGELOG file.
  • Added 1 missing contributor from v2.10.0 to the ! AUTHORS file.

PR Checklist

  • [x] Issue raised/referenced?
  • [ ] Tests updated/added?
  • [x] Documentation updated/added?
  • [x] package/CHANGELOG file updated?
  • [x] Is your name in package/AUTHORS? (If it is not, add it!)

Developers Certificate of Origin

I certify that this code contribution as described in the Developer Certificate of Origin, under the MDAnalysis LICENSE is my own original work.

  • [x] I agree to the Developer Certificate of Origin.

📚 Documentation preview 📚: https://mdanalysis--5139.org.readthedocs.build/en/5139/

Aryaman-Chaudhri avatar Nov 01 '25 07:11 Aryaman-Chaudhri

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 92.73%. Comparing base (5c7c480) to head (804c27d).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #5139      +/-   ##
===========================================
+ Coverage    92.68%   92.73%   +0.04%     
===========================================
  Files          180      180              
  Lines        22452    22452              
  Branches      3186     3186              
===========================================
+ Hits         20809    20820      +11     
- Misses        1169     1176       +7     
+ Partials       474      456      -18     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Nov 01 '25 07:11 codecov[bot]

Hi Irfan, thanks for the review! I've removed the stray comment and pushed the fix.

I also went through all the PRs listed in the 2.10.0 changelog and checked their authors against the contributor list. It looks like the 5 authors from the issue were the only ones missing.

I believe the list is complete now. Thanks again!

Aryaman-Chaudhri avatar Nov 01 '25 18:11 Aryaman-Chaudhri