search

qiime2 / docs

https://docs.qiime2.org
BSD 3-Clause "New" or "Revised" License
21 stars 58 forks source link

MAINT: updates `diversity.methods.alpha` to `pipelines.alpha` #489

Closed ChrisKeefe closed 3 years ago

ChrisKeefe commented 4 years ago

~Also, miscellaneous copy edits in PD Mice tutorial~ PD Mice tutorial edits have been moved to #488

ChrisKeefe commented 4 years ago

forum xref for diversity.methods->pipelines

thermokarst commented 4 years ago

Please move the misc copy edits to #488

thermokarst commented 3 years ago

@ChrisKeefe - please move the misc copy edits to #488.

thermokarst commented 3 years ago

Mark this as "ready for review" when you're all set, @ChrisKeefe.

ChrisKeefe commented 3 years ago

@thermokarst, I've added a couple of artifact-api copy edits to this PR. Do you want those here, in #488, or in a separate PR?

Once that question's handled, this will be ready for review.

thermokarst commented 3 years ago

Here makes sense to me.

To be clear, my request for you to remove copy edits wasn't a blanket statement for all copy edits, just for the PD Mice ones, since they A) had nothing to do with this PR, B) we already had a plan in place to handle the bulk updates for copy edits to the PD Mice tutorial, and C) collaborating on a git branch is a useful exercise for the lab.

thermokarst commented 3 years ago

When you're ready for review, please:

  1. Mark as "ready for review"
  2. Move this card into the "in review" column on the project board
  3. Unassign yourself

Thanks!

andrewsanchez commented 3 years ago

There was one minor correction I made to an output file name but other than that, LGTM! 🌮