MDAnalysis / UserGuide

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

fixes #201: update docs for CI/CD providers #291

Closed jandom closed 10 months ago

jandom commented 11 months ago

please suggest changes to the actual text/copy, I just wrote down 1-2 quick sentences

jandom commented 11 months ago

@orbeckst bump

jandom commented 10 months ago

This has gone a bit stale, and I don't want to continue working on this. I'm going to close – feel free to re-open/re-assign if there are any volunteers. Long-term open PRs are pretty bad in my book, they obstruct visibility from what's really needed.

IAlibay commented 10 months ago

Sorry about that, I didn't see any blocking reviews - I assumed this was just go to go as-is?

jandom commented 10 months ago

Unfortunately I believe pretty much everything here is on me. I don't have time to spend time writing everything at this point though.

I parsed this as pretty blocking, no? Did you mean that it's passable but will need a further touch-up from you?

IAlibay commented 10 months ago

Did you mean that it's passable but will need a further touch-up from you?

Yes, my point was "the details are all on me", which is what I meant regarding placeholders. Generally the view we take is that if we don't leave a blocking review then it's not blocking.

jandom commented 10 months ago

Gotcha, okay then let's resume – I've restored the branch

jandom commented 10 months ago

Rebased the branch onto develop to get this green again

jandom commented 10 months ago

Many thanks @IAlibay and @lilyminium – thankful to have this merged 🏆