search

MDAnalysis / UserGuide

User Guide for MDAnalysis
https://userguide.mdanalysis.org
22 stars 31 forks source link

Have the right project slug on the rtd.yaml action #323

Closed IAlibay closed 9 months ago

IAlibay commented 9 months ago

The action isn't as relevant now that RTD points back to the build by default, but it still helps new users know where the build is (rather than searching through check links).

:books: Documentation preview :books:: https://mdanalysis--323.org.readthedocs.build/en/323/

lilyminium commented 9 months ago

LGTM I think -- does it only update after merge? The top level comment still points to MDA

lilyminium commented 9 months ago

Or is it mdanalysisuserguide?

IAlibay commented 9 months ago

LGTM I think -- does it only update after merge? The top level comment still points to MDA

Yeah it's a pull_request_trigger workflow so it'll only pick up what's merged.

IAlibay commented 9 months ago

Or is it mdanalysisuserguide?

Might be, let me double check

IAlibay commented 9 months ago

Yeah my bad, it's the rtd project name not the repo name - fixed.