Skip to content

Remove RTD labeller#5288

Merged
orbeckst merged 1 commit into
developfrom
remove-rtd-labeller
May 30, 2026
Merged

Remove RTD labeller#5288
orbeckst merged 1 commit into
developfrom
remove-rtd-labeller

Conversation

@IAlibay
Copy link
Copy Markdown
Member

@IAlibay IAlibay commented Mar 7, 2026

Fixes #5287

LLM / AI generated code disclosure

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

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.


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

@IAlibay
Copy link
Copy Markdown
Member Author

IAlibay commented Mar 7, 2026

Actually needs installing the RTD gh app to the project. Any objections @MDAnalysis/coredevs ?

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.16%. Comparing base (2f240fb) to head (343b031).
⚠️ Report is 43 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #5288      +/-   ##
===========================================
+ Coverage    93.82%   94.16%   +0.33%     
===========================================
  Files          180      182       +2     
  Lines        22477    23816    +1339     
  Branches      3191     3490     +299     
===========================================
+ Hits         21090    22427    +1337     
- Misses         924      928       +4     
+ Partials       463      461       -2     

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

@orbeckst
Copy link
Copy Markdown
Member

orbeckst commented Mar 7, 2026 via email

Copy link
Copy Markdown
Member

@orbeckst orbeckst left a comment

Choose a reason for hiding this comment

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

Thanks for doing the maintenance/cleanup!

@orbeckst
Copy link
Copy Markdown
Member

@IAlibay can you please merge when you have the new GH app installed? You know better than I how to make this a smooth transition. Thanks!

@orbeckst
Copy link
Copy Markdown
Member

ping @IAlibay to merge this PR (or note that something else needs to be done)

@orbeckst
Copy link
Copy Markdown
Member

The Read The Docs Community app is now enabled for the whole org.

@orbeckst orbeckst mentioned this pull request May 30, 2026
8 tasks
@orbeckst
Copy link
Copy Markdown
Member

Let me cycle the PR to see if RTD works (see #5392 )

@orbeckst orbeckst closed this May 30, 2026
@orbeckst orbeckst reopened this May 30, 2026
@orbeckst
Copy link
Copy Markdown
Member

I enabled Show build overview in a comment.

@orbeckst orbeckst closed this May 30, 2026
@orbeckst
Copy link
Copy Markdown
Member

Let's see if the build overview appears.

@orbeckst orbeckst reopened this May 30, 2026
@orbeckst
Copy link
Copy Markdown
Member

RTD appears to be working, new preview appears, see above.

@orbeckst orbeckst merged commit b05a84b into develop May 30, 2026
50 of 68 checks passed
@orbeckst orbeckst deleted the remove-rtd-labeller branch May 30, 2026 17:07
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.

Remove RTD labelling action in favour of using the build preview directly from rtd

2 participants