Skip to content

Edit tutorial to accommodate da_barplot refactor#12

Merged
colinvwood merged 2 commits into
qiime2:mainfrom
Macabe222:da-barplot-rework
Jul 7, 2026
Merged

Edit tutorial to accommodate da_barplot refactor#12
colinvwood merged 2 commits into
qiime2:mainfrom
Macabe222:da-barplot-rework

Conversation

@Macabe222

Copy link
Copy Markdown
Contributor

Remove significance_threshold from tutorial since new da_barplot no longer allows for this.
Linked to qiime2/q2-composition#166
Addresses qiime2/q2-composition#156

@lizgehret lizgehret left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm 👍🏻

@github-project-automation github-project-automation Bot moved this to Ready for Merge in 2026.7 🐐 May 20, 2026
@lizgehret

Copy link
Copy Markdown
Member

this is ready for merge but i'll hold off until after we discuss the paired changes in composition

@colinvwood colinvwood self-assigned this Jul 7, 2026
@colinvwood colinvwood merged commit f241de7 into qiime2:main Jul 7, 2026
1 check was pending
@github-project-automation github-project-automation Bot moved this from Ready for Merge to Changelog Needed in 2026.7 🐐 Jul 7, 2026
@colinvwood colinvwood moved this from Changelog Needed to Completed in 2026.7 🐐 Jul 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Completed

Development

Successfully merging this pull request may close these issues.

4 participants